| 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: <20100809214852.GK3635@thunk.org> Date: Mon, 9 Aug 2010 17:48:52 -0400 From: Ted Ts'o <tytso@....edu> To: Linus Torvalds <torvalds@...ux-foundation.org> Cc: linux-ext4@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [GIT PULL] ext4 update for 2.6.36 On Sat, Aug 07, 2010 at 01:17:46PM -0700, Linus Torvalds wrote: > On Thu, Aug 5, 2010 at 9:23 PM, Theodore Ts'o <tytso@....edu> wrote: > > > > Which is hopefully what you want. If you want exactly what is shown > > below, you can also pull the "next" branch, which omits the merge, and > > then you can deal with the merge conflict by hand. The results should > > be the same either way. > > I did that just to see what it looked like. > > One annoyance: I now get > > fs/jbd2/transaction.c: In function ‘start_this_handle’: > fs/jbd2/transaction.c:98: warning: unused variable ‘ts’ > > due to that 'ts' thing only being used for the CONFIG_JBD2_DEBUG case. > Please fix. I'd suggest putting that CONFIG_JBD2_DEBUG code in a > helper function of its own, and declaring 'ts' there instead. > Suggested patch (but with a horrible helper function name) attached. This is what I have in the ext4 tree. I'll be sending a pull request in a day or two with this and maybe one or two other fixups if they arise. So far the post-integration testing I've done looks good, though. - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists