linux-kernel - Re: [PATCH] arch: uapi: asm: mman.h: Support MADV

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: <547D1C42.1010502@gmail.com>
Date:	Tue, 02 Dec 2014 09:56:18 +0800
From:	Chen Gang <gang.chen.5i5j@...il.com>
To:	Carlos O'Donell <carlos@...temhalted.org>,
	Geert Uytterhoeven <geert@...ux-m68k.org>
CC:	Max Filippov <jcmvbkbc@...il.com>,
	Minchan Kim <minchan@...nel.org>,
	"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
	"rth@...ddle.net" <rth@...ddle.net>,
	"ink@...assic.park.msu.ru" <ink@...assic.park.msu.ru>,
	"mattst88@...il.com" <mattst88@...il.com>,
	Ralf Baechle <ralf@...ux-mips.org>,
	"jejb@...isc-linux.org" <jejb@...isc-linux.org>,
	"deller@....de" <deller@....de>,
	"chris@...kel.net" <chris@...kel.net>,
	"linux-alpha@...r.kernel.org" <linux-alpha@...r.kernel.org>,
	"linux-mips@...ux-mips.org" <linux-mips@...ux-mips.org>,
	"linux-parisc@...r.kernel.org" <linux-parisc@...r.kernel.org>,
	"linux-xtensa@...ux-xtensa.org" <linux-xtensa@...ux-xtensa.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Arnd Bergmann <arnd@...db.de>
Subject: Re: [PATCH] arch: uapi: asm: mman.h: Support MADV_FREE for madvise()

On 12/2/14 06:01, Carlos O'Donell wrote:
> On Mon, Dec 1, 2014 at 4:35 PM, Geert Uytterhoeven <geert@...ux-m68k.org> wrote:
>> On Mon, Dec 1, 2014 at 9:52 PM, Max Filippov <jcmvbkbc@...il.com> wrote:
>>> On Mon, Dec 1, 2014 at 11:43 PM, Chen Gang <gang.chen.5i5j@...il.com> wrote:
>>>> At present, kernel supports madvise(MADV_FREE), so can benefit to all
>>>> related architectures (can grep MADV_WILLNEED or MADV_REMOVE in "arch/"
>>>> to know about all related architectures).
>>>
>>> A similar patch has been posted a while ago:
>>>
>>> http://www.spinics.net/lists/linux-mm/msg81538.html
>>

OK, thanks.


>> Would it be possible to use the same number everywhere?
> 

For current patch, I guess, we can use '8' for it, since MADV_FREE in
asm-generic is merged just a few days ago (which is not used by user
mode), and parisc has to use '8'.

And welcome the related member's ideas.


> Yes please. It's ridiculous that we still need patches like this.
> 
> I proposed unifying all this two years ago, but didn't follow up.
> 
> From glibc's perspective it would be simpler if we started using the
> same number everywhere.
> 
> http://www.spinics.net/lists/linux-api/msg02064.html
> 

For me, we can divide it into 2 steps:

 - Let MADV_FREE has the same value (about current patch).

 - Let all shared MADV_* to "asm-generic" (about next patch, although I
   am not quite sure whether it is executable).


Thanks.
-- 
Chen Gang

Open, share, and attitude like air, water, and life which God blessed
--
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/