linux-kernel - Re: [PATCH] jbd: change the field "b_cow_tid" of struct journal_head from type unsigned to tid

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAOQ4uxhr8M=+p7oWK6NevHNDm=E3qY3Aju8D0ZAc+D2M8aFG1A@mail.gmail.com>
Date:	Mon, 25 Jul 2011 17:19:29 +0300
From:	Amir Goldstein <amir73il@...il.com>
To:	Christoph Hellwig <hch@...radead.org>
Cc:	Wang Sheng-Hui <shhuiw@...il.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Jan Kara <jack@...e.cz>, linux-ext4@...r.kernel.org,
	linux-kernel@...r.kernel.org, Theodore Tso <tytso@....edu>
Subject: Re: [PATCH] jbd: change the field "b_cow_tid" of struct journal_head
 from type unsigned to tid_t

On Mon, Jul 25, 2011 at 4:44 PM, Christoph Hellwig <hch@...radead.org> wrote:
> On Mon, Jul 25, 2011 at 09:02:25PM +0800, Wang Sheng-Hui wrote:
>> The patch is against 3.0
>>
>> In the definition of struct journal_head, the comment for
>> the field "unsigned b_cow_tid" says the field tracks the
>> last transaction id in which this buffer has been cowed.
>
> The correct fix is to simply remove the field, as it's entirely unused.
>

It's part of the ext4 snapshot patches, which are making their way into mainline
as slowly as one could expect them to.
This patch was merged for 2.6.39, so more users will be able to test
ext4 snapshots
independent module without having to patch their kernels, so that the
patches get proper
QA before they arrive to mainline.

Cheers,
Amir.
--
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/