[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20171230015704.GE11757@thunk.org>
Date: Fri, 29 Dec 2017 20:57:04 -0500
From: Theodore Ts'o <tytso@....edu>
To: Andreas Dilger <adilger@...ger.ca>
Cc: linux-ext4 <linux-ext4@...r.kernel.org>,
Shuichi Ihara <sihara@....com>, Li Xi <lixi@....com>,
Wang Shilong <wshilong@....com>
Subject: Re: [PATCH] misc: add e2mmpstatus utility
On Fri, Dec 29, 2017 at 04:02:04PM -0700, Andreas Dilger wrote:
> > Why not include this into dumpe2fs? Maybe dumpe2fs -m to just print
> > the MMP information and include the MMP nodetime and time for either
> > the full dumpe2fs or dumpe2fs -h output?
>
> That would definitely be an option. The mmpstatus utility was written by
> DDN, and I'm passing it on after some code cleanup.
>
> I think it would be reasonable to include the other MMP fields into the
> dumpe2fs output, and then run the full MMP activity check if called with
> "-m" or named as "mmpstatus". I don't want dumpe2fs to be blocked on
> the MMP update checking all the time, since that can be slow.
SGTM.
> I'll see if I can get a patch out in the next couple of days.
Many thanks!!
Cheers,
- Ted
Powered by blists - more mailing lists