Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
openSUSE:Leap:42.1:Staging:B
util-linux-systemd
util-linux-fsck-implement-fsck-r_fd.patch
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File util-linux-fsck-implement-fsck-r_fd.patch of Package util-linux-systemd
From 07c09a29eaf359f75a8adf8bb0320f3f2d97c5cc Mon Sep 17 00:00:00 2001 From: Stanislav Brabec <sbrabec@suse.cz> Date: Mon, 27 Apr 2015 17:20:55 +0200 Subject: [PATCH] fsck: implement fsck -r {fd} Make possible sending of statistics to a dedicated file descriptor. Rationale: When UI is calling fsck from a remote terminal, fsck progress needs to be sent to stdout. It is mixed there with output of statistics, and it is impossible to parse the output to get the statistics. Now it will be possible e. g. with "fsck -C -r 3 /dev/sda1" Note: Code in if and else is intentionally partially duplicated. Current human readable output of floats does not conform to locale conventions, and may be changed in future. But we want to keep machine readable output exactly same as it is now. [kzak@redhat.com: - don't use report_stats_fd globally] Signed-off-by: Stanislav Brabec <sbrabec@suse.cz> Signed-off-by: Karel Zak <kzak@redhat.com> --- disk-utils/fsck.8 | 13 +++++++++-- disk-utils/fsck.c | 64 ++++++++++++++++++++++++++++++++++++++++++++----------- 2 files changed, 63 insertions(+), 14 deletions(-) Index: util-linux-2.25/disk-utils/fsck.8 =================================================================== --- util-linux-2.25.orig/disk-utils/fsck.8 +++ util-linux-2.25/disk-utils/fsck.8 @@ -7,7 +7,9 @@ fsck \- check and repair a Linux filesystem .SH SYNOPSIS .B fsck -.RB [ \-lrsAVRTMNP ] +.RB [ \-lsAVRTMNP ] +.RB [ \-r +.RI [ fd ]] .RB [ \-C .RI [ fd ]] .RB [ \-t @@ -104,13 +106,20 @@ multiple devices or for non-rotating dis devices when executed to check stacked devices (e.g.\& MD or DM) \(en this feature is not implemented yet. .TP -.B \-r +.BR \-r \ [ \fIfd\fR ] Report certain statistics for each fsck when it completes. These statistics include the exit status, the maximum run set size (in kilobytes), the elapsed all-clock time and the user and system CPU time used by the fsck run. For example: /dev/sda1: status 0, rss 92828, real 4.002804, user 2.677592, sys 0.86186 + +GUI front-ends may specify a file descriptor +.IR fd , +in which case the progress bar information will be sent to that file descriptor +in a machine parseable format. For example: + +/dev/sda1 0 92828 4.002804 2.677592 0.86186 .TP .B \-s Serialize Index: util-linux-2.25/disk-utils/fsck.c =================================================================== --- util-linux-2.25.orig/disk-utils/fsck.c +++ util-linux-2.25/disk-utils/fsck.c @@ -141,6 +141,7 @@ static int progress; static int progress_fd; static int force_all_parallel; static int report_stats; +static FILE *report_stats_file; static int num_running; static int max_running; @@ -582,16 +583,28 @@ static void print_stats(struct fsck_inst time_diff = (inst->end_time.tv_sec - inst->start_time.tv_sec) + (inst->end_time.tv_usec - inst->start_time.tv_usec) / 1E6; - fprintf(stdout, "%s: status %d, rss %ld, " - "real %f, user %d.%06d, sys %d.%06d\n", - fs_get_device(inst->fs), - inst->exit_status, - inst->rusage.ru_maxrss, - time_diff, - (int)inst->rusage.ru_utime.tv_sec, - (int)inst->rusage.ru_utime.tv_usec, - (int)inst->rusage.ru_stime.tv_sec, - (int)inst->rusage.ru_stime.tv_usec); + if (report_stats_file) + fprintf(report_stats_file, "%s %d %ld " + "%f %d.%06d %d.%06d\n", + fs_get_device(inst->fs), + inst->exit_status, + inst->rusage.ru_maxrss, + time_diff, + (int)inst->rusage.ru_utime.tv_sec, + (int)inst->rusage.ru_utime.tv_usec, + (int)inst->rusage.ru_stime.tv_sec, + (int)inst->rusage.ru_stime.tv_usec); + else + fprintf(stdout, "%s: status %d, rss %ld, " + "real %f, user %d.%06d, sys %d.%06d\n", + fs_get_device(inst->fs), + inst->exit_status, + inst->rusage.ru_maxrss, + time_diff, + (int)inst->rusage.ru_utime.tv_sec, + (int)inst->rusage.ru_utime.tv_usec, + (int)inst->rusage.ru_stime.tv_sec, + (int)inst->rusage.ru_stime.tv_usec); } /* @@ -1364,11 +1377,12 @@ static void __attribute__((__noreturn__) fputs(_(" -N do not execute, just show what would be done\n"), out); fputs(_(" -P check filesystems in parallel, including root\n"), out); fputs(_(" -R skip root filesystem; useful only with '-A'\n"), out); - fputs(_(" -r report statistics for each device checked\n"), out); + fputs(_(" -r [<fd>] report statistics for each device checked;\n" + " file descriptor is for GUIs\n"), out); fputs(_(" -s serialize the checking operations\n"), out); fputs(_(" -T do not show the title on startup\n"), out); fputs(_(" -t <type> specify filesystem types to be checked;\n" - " <type> is allowed to be a comma-separated list\n"), out); + " <type> is allowed to be a comma-separated list\n"), out); fputs(_(" -V explain what is being done\n"), out); fputs(_(" -? display this help and exit\n"), out); @@ -1392,6 +1406,7 @@ static void parse_argv(int argc, char *a int opt = 0; int opts_for_fsck = 0; struct sigaction sa; + int report_stats_fd = 0; /* * Set up signal action @@ -1498,6 +1513,21 @@ static void parse_argv(int argc, char *a break; case 'r': report_stats = 1; + if (arg[j+1]) { /* -r<fd> */ + report_stats_fd = string_to_int(arg+j+1); + if (report_stats_fd < 0) + report_stats_fd = 0; + else + goto next_arg; + } else if (i+1 < argc && *argv[i+1] != '-') { /* -r <fd> */ + report_stats_fd = string_to_int(argv[i+1]); + if (report_stats_fd < 0) + report_stats_fd = 0; + else { + ++i; + goto next_arg; + } + } break; case 's': serialize = 1; @@ -1536,6 +1566,16 @@ static void parse_argv(int argc, char *a opt = 0; } } + + /* Validate the report stats file descriptor to avoid disasters */ + if (report_stats_fd) { + report_stats_file = fdopen(report_stats_fd, "w"); + if (!report_stats_file) + err(FSCK_EX_ERROR, + _("invalid argument -r %d"), + report_stats_fd); + } + if (getenv("FSCK_FORCE_ALL_PARALLEL")) force_all_parallel++; if ((tmp = getenv("FSCK_MAX_INST")))
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor
