[PATCH 2/4] xfsprogs: Addition in debug option -d of xfs_spaceman

Dhruvesh Rathore adrscube at gmail.com
Wed Feb 11 10:17:09 CST 2015


On Wed, Feb 11, 2015 at 3:44 AM, Dave Chinner <david at fromorbit.com> wrote:
> On Thu, Jan 29, 2015 at 06:55:21PM +0530, Dhruvesh Rathore wrote:
>>
>> The xfs_spaceman utility did not have a a proper title to the content
>> displayed using -d (debug) option.
> .....
>>
>> @@ -332,20 +332,23 @@
>>  static int
>>  freesp_f(
>>       int             argc,
>>       char            **argv)
>>  {
>>       xfs_agnumber_t  agno;
>>
>>       if (!init(argc, argv))
>>               return 0;
>>
>> +     if (dumpflag)
>> +             printf("%8s %8s %8s\n", "agno", "agbno", "len");
>> +
>>       for (agno = 0; agno < file->geom.agcount; agno++)  {
>>               if (inaglist(agno))
>>                       scan_ag(agno);
>>       }
>>       if (histcount)
>>               printhist();
>
> Patch does not apply to the current tree. It's got stray whitespace
> and lines in it that are in the original source file.
>
> I've fixed this up for the moment, but please make sure your
> patches apply to a clean tree.
>

We apologize for the constant stray white space errors and will
make sure to make clean patches from now on.
Thank you for reworking and applying the patch to the current tree :)

Regards,
A-DRS



More information about the xfs mailing list