Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-12 Thread Davidlohr Bueso
On Sat, 2014-04-12 at 10:50 +0200, Manfred Spraul wrote: > On 04/11/2014 10:27 PM, Davidlohr Bueso wrote: > > On Fri, 2014-04-11 at 20:28 +0200, Manfred Spraul wrote: > >> Hi Davidlohr, > >> > >> On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: > >>> The default size for shmmax is, and always has

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-12 Thread Manfred Spraul
On 04/11/2014 10:27 PM, Davidlohr Bueso wrote: On Fri, 2014-04-11 at 20:28 +0200, Manfred Spraul wrote: Hi Davidlohr, On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small,

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-12 Thread Manfred Spraul
On 04/11/2014 10:27 PM, Davidlohr Bueso wrote: On Fri, 2014-04-11 at 20:28 +0200, Manfred Spraul wrote: Hi Davidlohr, On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small,

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-12 Thread Davidlohr Bueso
On Sat, 2014-04-12 at 10:50 +0200, Manfred Spraul wrote: On 04/11/2014 10:27 PM, Davidlohr Bueso wrote: On Fri, 2014-04-11 at 20:28 +0200, Manfred Spraul wrote: Hi Davidlohr, On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: The default size for shmmax is, and always has been, 32Mb.

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-11 Thread Davidlohr Bueso
On Fri, 2014-04-11 at 13:27 -0700, Davidlohr Bueso wrote: > On Fri, 2014-04-11 at 20:28 +0200, Manfred Spraul wrote: > > Your patch disables checking shmmax, shmall *AND* checking for SHMMIN. > > Right, if shmmax is 0, then there's no point checking for shmmin, > otherwise we'd always end up

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-11 Thread Davidlohr Bueso
On Fri, 2014-04-11 at 20:28 +0200, Manfred Spraul wrote: > Hi Davidlohr, > > On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: > > The default size for shmmax is, and always has been, 32Mb. > > Today, in the XXI century, it seems that this value is rather small, > > making users have to increase it

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-11 Thread Manfred Spraul
Hi Davidlohr, On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small, making users have to increase it via sysctl, which can cause unnecessary work and userspace application

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-11 Thread Manfred Spraul
Hi Davidlohr, On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small, making users have to increase it via sysctl, which can cause unnecessary work and userspace application

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-11 Thread Davidlohr Bueso
On Fri, 2014-04-11 at 20:28 +0200, Manfred Spraul wrote: Hi Davidlohr, On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small, making users have to increase it via

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-11 Thread Davidlohr Bueso
On Fri, 2014-04-11 at 13:27 -0700, Davidlohr Bueso wrote: On Fri, 2014-04-11 at 20:28 +0200, Manfred Spraul wrote: Your patch disables checking shmmax, shmall *AND* checking for SHMMIN. Right, if shmmax is 0, then there's no point checking for shmmin, otherwise we'd always end up returning

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-06 Thread Davidlohr Bueso
On Sun, 2014-04-06 at 08:42 +0200, Manfred Spraul wrote: > Hi, > > On 04/05/2014 08:24 PM, KOSAKI Motohiro wrote: > > On Fri, Apr 4, 2014 at 1:00 AM, Davidlohr Bueso wrote: > >> I don't think it makes much sense to set unlimited for both 0 and > >> ULONG_MAX, that would probably just create even

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-06 Thread Manfred Spraul
Hi, On 04/05/2014 08:24 PM, KOSAKI Motohiro wrote: On Fri, Apr 4, 2014 at 1:00 AM, Davidlohr Bueso wrote: I don't think it makes much sense to set unlimited for both 0 and ULONG_MAX, that would probably just create even more confusion. I agree. Unlimited was INT_MAX since 0.99.10 and

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-06 Thread Manfred Spraul
Hi, On 04/05/2014 08:24 PM, KOSAKI Motohiro wrote: On Fri, Apr 4, 2014 at 1:00 AM, Davidlohr Bueso davidl...@hp.com wrote: I don't think it makes much sense to set unlimited for both 0 and ULONG_MAX, that would probably just create even more confusion. I agree. Unlimited was INT_MAX since

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-06 Thread Davidlohr Bueso
On Sun, 2014-04-06 at 08:42 +0200, Manfred Spraul wrote: Hi, On 04/05/2014 08:24 PM, KOSAKI Motohiro wrote: On Fri, Apr 4, 2014 at 1:00 AM, Davidlohr Bueso davidl...@hp.com wrote: I don't think it makes much sense to set unlimited for both 0 and ULONG_MAX, that would probably just create

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-05 Thread KOSAKI Motohiro
On Fri, Apr 4, 2014 at 1:00 AM, Davidlohr Bueso wrote: > On Thu, 2014-04-03 at 19:39 -0400, KOSAKI Motohiro wrote: >> On Thu, Apr 3, 2014 at 3:50 PM, Davidlohr Bueso wrote: >> > On Thu, 2014-04-03 at 21:02 +0200, Manfred Spraul wrote: >> >> Hi Davidlohr, >> >> >> >> On 04/03/2014 02:20 AM,

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-05 Thread KOSAKI Motohiro
On Fri, Apr 4, 2014 at 1:00 AM, Davidlohr Bueso davidl...@hp.com wrote: On Thu, 2014-04-03 at 19:39 -0400, KOSAKI Motohiro wrote: On Thu, Apr 3, 2014 at 3:50 PM, Davidlohr Bueso davidl...@hp.com wrote: On Thu, 2014-04-03 at 21:02 +0200, Manfred Spraul wrote: Hi Davidlohr, On 04/03/2014

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread Davidlohr Bueso
On Thu, 2014-04-03 at 19:39 -0400, KOSAKI Motohiro wrote: > On Thu, Apr 3, 2014 at 3:50 PM, Davidlohr Bueso wrote: > > On Thu, 2014-04-03 at 21:02 +0200, Manfred Spraul wrote: > >> Hi Davidlohr, > >> > >> On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: > >> > The default size for shmmax is, and

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread KOSAKI Motohiro
> This change allows Linux to treat shm just as regular anonymous memory. > One important difference between them, though, is handling out-of-memory > conditions: as opposed to regular anon memory, the OOM killer will not > kill processes that are hogging memory through shm, allowing users to >

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread KOSAKI Motohiro
On Thu, Apr 3, 2014 at 3:50 PM, Davidlohr Bueso wrote: > On Thu, 2014-04-03 at 21:02 +0200, Manfred Spraul wrote: >> Hi Davidlohr, >> >> On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: >> > The default size for shmmax is, and always has been, 32Mb. >> > Today, in the XXI century, it seems that

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread KOSAKI Motohiro
On Wed, Apr 2, 2014 at 8:20 PM, Davidlohr Bueso wrote: > The default size for shmmax is, and always has been, 32Mb. > Today, in the XXI century, it seems that this value is rather small, > making users have to increase it via sysctl, which can cause > unnecessary work and userspace application

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread Davidlohr Bueso
On Thu, 2014-04-03 at 21:02 +0200, Manfred Spraul wrote: > Hi Davidlohr, > > On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: > > The default size for shmmax is, and always has been, 32Mb. > > Today, in the XXI century, it seems that this value is rather small, > > making users have to increase it

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread Manfred Spraul
Hi Davidlohr, On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small, making users have to increase it via sysctl, which can cause unnecessary work and userspace application

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread Kamezawa Hiroyuki
(2014/04/03 9:20), Davidlohr Bueso wrote: The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small, making users have to increase it via sysctl, which can cause unnecessary work and userspace application workarounds[1].

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread Kamezawa Hiroyuki
(2014/04/03 9:20), Davidlohr Bueso wrote: The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small, making users have to increase it via sysctl, which can cause unnecessary work and userspace application workarounds[1].

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread Manfred Spraul
Hi Davidlohr, On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small, making users have to increase it via sysctl, which can cause unnecessary work and userspace application

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread Davidlohr Bueso
On Thu, 2014-04-03 at 21:02 +0200, Manfred Spraul wrote: Hi Davidlohr, On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small, making users have to increase it via

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread KOSAKI Motohiro
On Wed, Apr 2, 2014 at 8:20 PM, Davidlohr Bueso davidl...@hp.com wrote: The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small, making users have to increase it via sysctl, which can cause unnecessary work and userspace

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread KOSAKI Motohiro
On Thu, Apr 3, 2014 at 3:50 PM, Davidlohr Bueso davidl...@hp.com wrote: On Thu, 2014-04-03 at 21:02 +0200, Manfred Spraul wrote: Hi Davidlohr, On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread KOSAKI Motohiro
This change allows Linux to treat shm just as regular anonymous memory. One important difference between them, though, is handling out-of-memory conditions: as opposed to regular anon memory, the OOM killer will not kill processes that are hogging memory through shm, allowing users to

Re: [PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-03 Thread Davidlohr Bueso
On Thu, 2014-04-03 at 19:39 -0400, KOSAKI Motohiro wrote: On Thu, Apr 3, 2014 at 3:50 PM, Davidlohr Bueso davidl...@hp.com wrote: On Thu, 2014-04-03 at 21:02 +0200, Manfred Spraul wrote: Hi Davidlohr, On 04/03/2014 02:20 AM, Davidlohr Bueso wrote: The default size for shmmax is, and

[PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-02 Thread Davidlohr Bueso
The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small, making users have to increase it via sysctl, which can cause unnecessary work and userspace application workarounds[1]. Instead of choosing yet another arbitrary value,

[PATCH] ipc,shm: disable shmmax and shmall by default

2014-04-02 Thread Davidlohr Bueso
The default size for shmmax is, and always has been, 32Mb. Today, in the XXI century, it seems that this value is rather small, making users have to increase it via sysctl, which can cause unnecessary work and userspace application workarounds[1]. Instead of choosing yet another arbitrary value,