linux-kernel - Re: remap_file

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: <20140519143837.GA5228@infradead.org>
Date:	Mon, 19 May 2014 07:38:37 -0700
From:	Christoph Hellwig <hch@...radead.org>
To:	"Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com>
Cc:	Kenny Simpson <theonetruekenny@...il.com>,
	Michal Hocko <mhocko@...e.cz>, linux-kernel@...r.kernel.org,
	Andrew Morton <akpm@...ux-foundation.org>,
	Peter Zijlstra <peterz@...radead.org>,
	Dave Jones <davej@...hat.com>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Armin Rigo <arigo@...es.org>
Subject: Re: remap_file_pages() use

On Mon, May 19, 2014 at 05:35:40PM +0300, Kirill A. Shutemov wrote:
> >From functional POV, emulation *should* be identical to original
> remap_file_pages(), but slower. It would be nice, if you test it early.
> 
> It's not clear yet how long emulation will be there.

Stop right there.  We found out about two real life users of
remap_file_pages() already, without even committing the patches to warn
about using it to any tree.

I think at this point the whole idea of removing the API should be dead
on the floor, as we do not needlessly break userspace programs.

If we can get rid of the ugly guts and provide a good enough emulation
that the user won't cry I'd love to get rid of this cruft, but even
that doesn't look certain yet.

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