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: <20130131165406.GA14486@thunk.org> Date: Thu, 31 Jan 2013 11:54:06 -0500 From: Theodore Ts'o <tytso@....edu> To: Dmitry Monakhov <dmonakhov@...nvz.org> Cc: Jan Kara <jack@...e.cz>, linux-ext4@...r.kernel.org Subject: Re: [PATCH 04/12] ext4: Disable merging of uninitialized extents On Thu, Jan 31, 2013 at 11:47:23AM +0400, Dmitry Monakhov wrote: > OK I've found it. I'm a bit disappointed, it is even not a race > condition, but simple corruption. > Patch is available here: http://article.gmane.org/gmane.comp.file-systems.ext4/36762 > link for sain mailer client: <1359617098-18451-1-git-send-email-dmonakhov@...nvz.org> > After this bug was fixed it is safe to apply both Jan's patches: > [PATCH 04/12] ext4: Disable merging of uninitialized extents > [PATCH 05/12] ext4: Remove unnecessary wait for extent conversion in ext4_fallocate() > At least it survives after all my tests. Thanks for finding it! As folks have probably noticed the dev branch and linux-next already has all of the other patches (except for #3, which we've agreed isn't needed since the original code was correct as-is). I'll work on reviewing your patch and then applying the last three, and then over the weekend I'll push the master branch up, thus casting all of the patches currently in the tree into stone. > BTW: It is appeared that ext4_debug() infrastructure is almost unusable > because based on printk() instead of light-wait event tracing infrastructure. > I'm now work on patch-set which fix that. Yes, I haven't used the ext4_debug infrastructure in quite some time. As I've needed to do debugging I've added new tracepoints, and in some cases I hadn't gotten around to removing the old ext4_debug code, on the assumption that maybe someone else was still using it, but mostly because I was lazy. That would be a great thing to do. - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists