linux-ext4 - Re: [GIT PULL] ext4 bugfixes

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 for Android: free password hash cracker in your pocket

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

Date:	Thu, 24 Jul 2008 16:03:16 -0400
From:	Christoph Hellwig <hch@...radead.org>
To:	Theodore Tso <tytso@....edu>,
	Christoph Hellwig <hch@...radead.org>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	linux-ext4@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [GIT PULL] ext4 bugfixes

On Tue, Jul 15, 2008 at 08:46:52AM -0400, Theodore Tso wrote:
> On Tue, Jul 15, 2008 at 12:37:18AM -0400, Christoph Hellwig wrote:
> > And for this one I haven't even heard back on my review comments.  None
> > of them is a merge blocker so it's not critical, but I find it quite odd
> > that the pull request is coming before even replying to the review
> > comments.
> 
> Can you resend your e-mail, please?  I can't seem to find it in my
> inbox.  Thanks!!

It was just a paragraph in the mail also commenting on the previously
core parts:

> Also the code still deals with the !buffer_mapped and no buffers on page
> cases all over which isn't needed anymore with ->page_mkwrite
> implemented.
> Similarly the !get_block case in mpage_da_writepages doesn't make any
> sense - it's never used and if people would want to use
> generic_writepages they could trivially just call it directly.

I've got a comment from Jan that the no buffers case can still happen,
so that can be removed.

Any chance you could also look into the kerneldoc comments for the newly
exported fs/mpage.c routines as commented on the updated patch?
--
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