xfs
[Top] [All Lists]

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

To: Dhruvesh Rathore <adrscube@xxxxxxxxx>
Subject: Re: [PATCH 2/4] xfsprogs: Addition in debug option -d of xfs_spaceman
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Wed, 11 Feb 2015 09:14:21 +1100
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <54ca34c6.4696420a.0f32.4513@xxxxxxxxxxxxx>
References: <54ca34c6.4696420a.0f32.4513@xxxxxxxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
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.

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>