Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread John Hubbard
On 12/02/2017 02:19 PM, Matthew Wilcox wrote: > On Sat, Dec 02, 2017 at 07:49:20PM +0100, Jann Horn wrote: >> On Sat, Dec 2, 2017 at 4:05 PM, Matthew Wilcox wrote: >>> On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: [...] > > Maybe that should be up

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread John Hubbard
On 12/02/2017 02:19 PM, Matthew Wilcox wrote: > On Sat, Dec 02, 2017 at 07:49:20PM +0100, Jann Horn wrote: >> On Sat, Dec 2, 2017 at 4:05 PM, Matthew Wilcox wrote: >>> On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: [...] > > Maybe that should be up front rather than

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread John Hubbard
On 12/02/2017 02:19 PM, Matthew Wilcox wrote: > On Sat, Dec 02, 2017 at 07:49:20PM +0100, Jann Horn wrote: >> On Sat, Dec 2, 2017 at 4:05 PM, Matthew Wilcox wrote: >>> On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: MAP_FIXED has been widely used

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread John Hubbard
On 12/02/2017 02:19 PM, Matthew Wilcox wrote: > On Sat, Dec 02, 2017 at 07:49:20PM +0100, Jann Horn wrote: >> On Sat, Dec 2, 2017 at 4:05 PM, Matthew Wilcox wrote: >>> On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: MAP_FIXED has been widely used for a very long time,

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread Matthew Wilcox
On Sat, Dec 02, 2017 at 07:49:20PM +0100, Jann Horn wrote: > On Sat, Dec 2, 2017 at 4:05 PM, Matthew Wilcox wrote: > > On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: > >> MAP_FIXED has been widely used for a very long time, yet the man > >> page still

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread Matthew Wilcox
On Sat, Dec 02, 2017 at 07:49:20PM +0100, Jann Horn wrote: > On Sat, Dec 2, 2017 at 4:05 PM, Matthew Wilcox wrote: > > On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: > >> MAP_FIXED has been widely used for a very long time, yet the man > >> page still claims that "the use

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread Jann Horn
On Sat, Dec 2, 2017 at 4:05 PM, Matthew Wilcox wrote: > On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: >> MAP_FIXED has been widely used for a very long time, yet the man >> page still claims that "the use of this option is discouraged". > > I think

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread Jann Horn
On Sat, Dec 2, 2017 at 4:05 PM, Matthew Wilcox wrote: > On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: >> MAP_FIXED has been widely used for a very long time, yet the man >> page still claims that "the use of this option is discouraged". > > I think we should continue to

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread Matthew Wilcox
On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: > MAP_FIXED has been widely used for a very long time, yet the man > page still claims that "the use of this option is discouraged". I think we should continue to discourage the use of this option, but I'm going to include

Re: [PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-02 Thread Matthew Wilcox
On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote: > MAP_FIXED has been widely used for a very long time, yet the man > page still claims that "the use of this option is discouraged". I think we should continue to discourage the use of this option, but I'm going to include

[PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-01 Thread john . hubbard
From: John Hubbard MAP_FIXED has been widely used for a very long time, yet the man page still claims that "the use of this option is discouraged". The documentation assumes that "less portable" == "must be discouraged". Instead of discouraging something that is so useful

[PATCH] mmap.2: MAP_FIXED is no longer discouraged

2017-12-01 Thread john . hubbard
From: John Hubbard MAP_FIXED has been widely used for a very long time, yet the man page still claims that "the use of this option is discouraged". The documentation assumes that "less portable" == "must be discouraged". Instead of discouraging something that is so useful and widely used,