linux-kernel - Re: [PATCH] jfs: clear PAGECACHE_TAG

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] [day] [month] [year] [list]

Date:	Sat, 29 Dec 2007 09:20:37 -0600
From:	Dave Kleikamp <shaggy@...ux.vnet.ibm.com>
To:	Fengguang Wu <wfg@...l.ustc.edu.cn>
Cc:	Peter Zijlstra <peterz@...radead.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	jfs-discussion@...ts.sourceforge.net,
	Nick Piggin <nickpiggin@...oo.com.au>
Subject: Re: [PATCH] jfs: clear PAGECACHE_TAG_DIRTY for no-write pages


On Sat, 2007-12-29 at 13:51 +0800, Fengguang Wu wrote:
> On Fri, Dec 28, 2007 at 10:50:59PM -0600, Dave Kleikamp wrote:
> > 
> > On Sat, 2007-12-29 at 10:21 +0800, Fengguang Wu wrote:

> > > This line should be moved below:
> > > > +	set_page_writeback(page);
> > 
> > No.  set_page_writeback() needs to be called before submit_bio() is
> > called.
> 
> Ah yes.
> 
> > I don't think there is any harm in calling set_page_writeback(),
> > redirty_page_for_writeback() and end_page_writeback() in the case where
> > there is no I/O to submit, and some dirty data cannot be written.  It is
> > consistent with what happens in __block_write_full_page().
> > 
> > It's also possible that some part of the page was written, and another
> > part cannot be, causing the page to be redirtied.
> 
> You are right. I revisited the code and there's nothing wrong with
> your patch :-)

Fengguang,

Thank you for all your help with this bug.  I've been on vacation and
hadn't had much time to look at it.  We'd probably still be scratching
our heads wondering where the problem was without your hard work.

Shaggy
> 
> Regards,
> Fengguang
> 
-- 
David Kleikamp
IBM Linux Technology Center

--
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/