lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <alpine.LFD.2.00.1405201150180.2162@localhost.localdomain> Date: Tue, 20 May 2014 11:56:53 +0200 (CEST) From: Lukáš Czerner <lczerner@...hat.com> To: Nikola Ciprich <nikola.ciprich@...uxbox.cz> cc: "Theodore Ts'o" <tytso@....edu>, "Darrick J. Wong" <darrick.wong@...cle.com>, linux-ext4@...r.kernel.org Subject: Re: [PATCH] ext4: add sysfs entry showing whether the fs contains errors On Tue, 20 May 2014, Nikola Ciprich wrote: > Date: Tue, 20 May 2014 11:47:41 +0200 > From: Nikola Ciprich <nikola.ciprich@...uxbox.cz> > To: Lukáš Czerner <lczerner@...hat.com> > Cc: Theodore Ts'o <tytso@....edu>, Darrick J. Wong <darrick.wong@...cle.com>, > linux-ext4@...r.kernel.org, Nikola Ciprich <nikola.ciprich@...uxbox.cz> > Subject: Re: [PATCH] ext4: add sysfs entry showing whether the fs contains > errors > > Hello Lukáš, > > just wanted to ask about the status of this patch.. it didn't make it > to git yet, did it? Do you plan to make some changes to it? I'm eager > to test it ;-) No it did not and it would not given that there was not a merge window since then. I've been distracted by other work, but I'll put you in cc when I have a new patch - yes I plan to change the format of the output to be more compliant with sysfs rules. -Lukas > > cheers > > nik > > > On Wed, May 07, 2014 at 10:44:34AM +0200, Lukáš Czerner wrote: > > On Tue, 6 May 2014, Theodore Ts'o wrote: > > > > > Date: Tue, 6 May 2014 15:23:26 -0400 > > > From: Theodore Ts'o <tytso@....edu> > > > To: Darrick J. Wong <darrick.wong@...cle.com> > > > Cc: Lukas Czerner <lczerner@...hat.com>, linux-ext4@...r.kernel.org, > > > nikola.ciprich@...uxbox.cz > > > Subject: Re: [PATCH] ext4: add sysfs entry showing whether the fs contains > > > errors > > > > > > On Tue, May 06, 2014 at 10:43:01AM -0700, Darrick J. Wong wrote: > > > > > > > > Minor complaint: "contains" makes me think that cat'ing that file will return > > > > either 0 or 1, not a string of error text. Perhaps we could shorten it to > > > > /sys/fs/ext4/sda/errors ? > > > > > > What I'd suggest doing is simply calling it errors_count, and > > > returning s_error_count. While we're at it, we could also return > > > s_first_error_time and s_last_error_time as well, since I imagine > > > those would could be quite useful for someone trying to create a > > > system health monitoring daemon. > > > > > > Cheers, > > > > > > - Ted > > > > Right now I am using the same function which prints this information > > to the log. So I am going to use simpler output for the sysfs. > > > > <error count> <first time> <function>:<line> <last time> <function>:<line> > > > > That should be easy to parse. I think that just calling it errors is > > good enough. > > > > Thanks! > > -Lukas > > > >
Powered by blists - more mailing lists