linux-kernel - Re: [RFC PATCH] s390: mm: rmap: Transfer storage key to struct page under the page lock

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: <alpine.LSU.2.00.1204181005500.1811@eggly.anvils>
Date:	Wed, 18 Apr 2012 10:09:24 -0700 (PDT)
From:	Hugh Dickins <hughd@...gle.com>
To:	Mel Gorman <mgorman@...e.de>
cc:	Martin Schwidefsky <schwidefsky@...ibm.com>,
	Heiko Carstens <heiko.carstens@...ibm.com>,
	Rik van Riel <riel@...hat.com>,
	Ken Chen <kenchen@...gle.com>, Linux-MM <linux-mm@...ck.org>,
	Linux-S390 <linux-s390@...r.kernel.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [RFC PATCH] s390: mm: rmap: Transfer storage key to struct page
 under the page lock

On Wed, 18 Apr 2012, Mel Gorman wrote:
> On Tue, Apr 17, 2012 at 08:52:21PM -0700, Hugh Dickins wrote:
> > 
> > It's a no-brainer workaround: patch and more explanation below.  I
> > can double-fix it if you prefer, but the one-liner appeals more to me.

(I became much happier about fixing it from this end, once I understood
how the situation came about.  What I had disliked with yours, was an
admitted ugly patch, when we didn't even understand the root cause.)

> 
> Ok, fair enough. While I think swapper space will eventually use the dirty
> tag information that day is not today.

Yes, it's never been self-evident to me, why swap should not participate
in any of the dirty writeback stuff.  But we've got along for years that
way, and as you say, won't be changing it right now.

> Acked-by: Mel Gorman <mgorman@...e.de>
> 
> I've sent a kernel based on this patch to the s390 folk that originally
> reported the bug. Hopefully they'll test and get back to me in a few
> days.

Thanks - I'll leave it at that for the moment, expecting to hear back.

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