Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-20 Thread Kees Cook
On Thu, Jul 20, 2017 at 5:34 AM, Eric W. Biederman wrote: > Ingo Molnar writes: > >> * Andrew Morton wrote: >> >>> On Wed, 19 Jul 2017 15:54:27 -0700 Davidlohr Bueso >>> wrote: >>> >>> > On Wed, 19 Jul

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-20 Thread Kees Cook
On Thu, Jul 20, 2017 at 5:34 AM, Eric W. Biederman wrote: > Ingo Molnar writes: > >> * Andrew Morton wrote: >> >>> On Wed, 19 Jul 2017 15:54:27 -0700 Davidlohr Bueso >>> wrote: >>> >>> > On Wed, 19 Jul 2017, Andrew Morton wrote: >>> > >>> > >I do rather dislike these conversions from the

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-20 Thread Eric W. Biederman
Ingo Molnar writes: > * Andrew Morton wrote: > >> On Wed, 19 Jul 2017 15:54:27 -0700 Davidlohr Bueso wrote: >> >> > On Wed, 19 Jul 2017, Andrew Morton wrote: >> > >> > >I do rather dislike these conversions from the point of

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-20 Thread Eric W. Biederman
Ingo Molnar writes: > * Andrew Morton wrote: > >> On Wed, 19 Jul 2017 15:54:27 -0700 Davidlohr Bueso wrote: >> >> > On Wed, 19 Jul 2017, Andrew Morton wrote: >> > >> > >I do rather dislike these conversions from the point of view of >> > >performance overhead and general code bloat. But I

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-20 Thread Ingo Molnar
* Andrew Morton wrote: > On Wed, 19 Jul 2017 15:54:27 -0700 Davidlohr Bueso wrote: > > > On Wed, 19 Jul 2017, Andrew Morton wrote: > > > > >I do rather dislike these conversions from the point of view of > > >performance overhead and general code

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-20 Thread Ingo Molnar
* Andrew Morton wrote: > On Wed, 19 Jul 2017 15:54:27 -0700 Davidlohr Bueso wrote: > > > On Wed, 19 Jul 2017, Andrew Morton wrote: > > > > >I do rather dislike these conversions from the point of view of > > >performance overhead and general code bloat. But I seem to have lost > > >that

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-19 Thread Kees Cook
On Wed, Jul 19, 2017 at 4:20 PM, Kees Cook wrote: > On Wed, Jul 19, 2017 at 4:11 PM, Davidlohr Bueso wrote: >> May I suggest using mmtests with the following config file: >> >>

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-19 Thread Kees Cook
On Wed, Jul 19, 2017 at 4:20 PM, Kees Cook wrote: > On Wed, Jul 19, 2017 at 4:11 PM, Davidlohr Bueso wrote: >> May I suggest using mmtests with the following config file: >> >>

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-19 Thread Kees Cook
On Wed, Jul 19, 2017 at 4:11 PM, Davidlohr Bueso wrote: > On Wed, 19 Jul 2017, Andrew Morton wrote: > >> On Wed, 19 Jul 2017 15:54:27 -0700 Davidlohr Bueso >> wrote: >> >>> On Wed, 19 Jul 2017, Andrew Morton wrote: >>> >>> >I do rather dislike these

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-19 Thread Kees Cook
On Wed, Jul 19, 2017 at 4:11 PM, Davidlohr Bueso wrote: > On Wed, 19 Jul 2017, Andrew Morton wrote: > >> On Wed, 19 Jul 2017 15:54:27 -0700 Davidlohr Bueso >> wrote: >> >>> On Wed, 19 Jul 2017, Andrew Morton wrote: >>> >>> >I do rather dislike these conversions from the point of view of >>>

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-19 Thread Davidlohr Bueso
On Wed, 19 Jul 2017, Andrew Morton wrote: On Wed, 19 Jul 2017 15:54:27 -0700 Davidlohr Bueso wrote: On Wed, 19 Jul 2017, Andrew Morton wrote: >I do rather dislike these conversions from the point of view of >performance overhead and general code bloat. But I seem to have

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-19 Thread Davidlohr Bueso
On Wed, 19 Jul 2017, Andrew Morton wrote: On Wed, 19 Jul 2017 15:54:27 -0700 Davidlohr Bueso wrote: On Wed, 19 Jul 2017, Andrew Morton wrote: >I do rather dislike these conversions from the point of view of >performance overhead and general code bloat. But I seem to have lost >that

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-19 Thread Andrew Morton
On Wed, 19 Jul 2017 15:54:27 -0700 Davidlohr Bueso wrote: > On Wed, 19 Jul 2017, Andrew Morton wrote: > > >I do rather dislike these conversions from the point of view of > >performance overhead and general code bloat. But I seem to have lost > >that struggle and I don't

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-19 Thread Andrew Morton
On Wed, 19 Jul 2017 15:54:27 -0700 Davidlohr Bueso wrote: > On Wed, 19 Jul 2017, Andrew Morton wrote: > > >I do rather dislike these conversions from the point of view of > >performance overhead and general code bloat. But I seem to have lost > >that struggle and I don't think any of these are

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-19 Thread Davidlohr Bueso
On Wed, 19 Jul 2017, Andrew Morton wrote: I do rather dislike these conversions from the point of view of performance overhead and general code bloat. But I seem to have lost that struggle and I don't think any of these are fastpath(?). Well, since we now have fd25d19 (locking/refcount:

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-19 Thread Davidlohr Bueso
On Wed, 19 Jul 2017, Andrew Morton wrote: I do rather dislike these conversions from the point of view of performance overhead and general code bloat. But I seem to have lost that struggle and I don't think any of these are fastpath(?). Well, since we now have fd25d19 (locking/refcount:

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-19 Thread Andrew Morton
On Sun, 09 Jul 2017 16:59:55 -0500 ebied...@xmission.com (Eric W. Biederman) wrote: > Elena Reshetova writes: > > > refcount_t type and corresponding API should be > > used instead of atomic_t when the variable is used as > > a reference counter. This allows to avoid

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-19 Thread Andrew Morton
On Sun, 09 Jul 2017 16:59:55 -0500 ebied...@xmission.com (Eric W. Biederman) wrote: > Elena Reshetova writes: > > > refcount_t type and corresponding API should be > > used instead of atomic_t when the variable is used as > > a reference counter. This allows to avoid accidental > > refcounter

RE: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-12 Thread Reshetova, Elena
> "Reshetova, Elena" writes: > > >> "Reshetova, Elena" writes: > >> > >> >> "Reshetova, Elena" writes: > >> >> > >> >> 2>> Elena Reshetova writes: > >> >> >> > >> >> >> > refcount_t

RE: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-12 Thread Reshetova, Elena
> "Reshetova, Elena" writes: > > >> "Reshetova, Elena" writes: > >> > >> >> "Reshetova, Elena" writes: > >> >> > >> >> 2>> Elena Reshetova writes: > >> >> >> > >> >> >> > refcount_t type and corresponding API should be > >> >> >> > used instead of atomic_t when the variable is used as > >>

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Eric W. Biederman
"Reshetova, Elena" writes: >> "Reshetova, Elena" writes: >> >> >> "Reshetova, Elena" writes: >> >> >> >> 2>> Elena Reshetova writes: >> >> >> >> >> >> > refcount_t type and

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Eric W. Biederman
"Reshetova, Elena" writes: >> "Reshetova, Elena" writes: >> >> >> "Reshetova, Elena" writes: >> >> >> >> 2>> Elena Reshetova writes: >> >> >> >> >> >> > refcount_t type and corresponding API should be >> >> >> > used instead of atomic_t when the variable is used as >> >> >> > a reference

RE: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Reshetova, Elena
> "Reshetova, Elena" writes: > > >> "Reshetova, Elena" writes: > >> > >> 2>> Elena Reshetova writes: > >> >> > >> >> > refcount_t type and corresponding API should be > >> >> > used instead of atomic_t when the

RE: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Reshetova, Elena
> "Reshetova, Elena" writes: > > >> "Reshetova, Elena" writes: > >> > >> 2>> Elena Reshetova writes: > >> >> > >> >> > refcount_t type and corresponding API should be > >> >> > used instead of atomic_t when the variable is used as > >> >> > a reference counter. This allows to avoid accidental

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Eric W. Biederman
"Reshetova, Elena" writes: >> "Reshetova, Elena" writes: >> >> 2>> Elena Reshetova writes: >> >> >> >> > refcount_t type and corresponding API should be >> >> > used instead of atomic_t when the variable is used

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Eric W. Biederman
"Reshetova, Elena" writes: >> "Reshetova, Elena" writes: >> >> 2>> Elena Reshetova writes: >> >> >> >> > refcount_t type and corresponding API should be >> >> > used instead of atomic_t when the variable is used as >> >> > a reference counter. This allows to avoid accidental >> >> >

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Eric W. Biederman
Alexey Dobriyan writes: > On Mon, Jul 10, 2017 at 11:37 AM, Eric W. Biederman > wrote: >> "Reshetova, Elena" writes: >> >> 2>> Elena Reshetova writes: > refcount_t type and

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Eric W. Biederman
Alexey Dobriyan writes: > On Mon, Jul 10, 2017 at 11:37 AM, Eric W. Biederman > wrote: >> "Reshetova, Elena" writes: >> >> 2>> Elena Reshetova writes: > refcount_t type and corresponding API should be > used instead of atomic_t when the variable is used as > a reference

RE: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Reshetova, Elena
> "Reshetova, Elena" writes: > > 2>> Elena Reshetova writes: > >> > >> > refcount_t type and corresponding API should be > >> > used instead of atomic_t when the variable is used as > >> > a reference counter. This allows to avoid accidental

RE: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Reshetova, Elena
> "Reshetova, Elena" writes: > > 2>> Elena Reshetova writes: > >> > >> > refcount_t type and corresponding API should be > >> > used instead of atomic_t when the variable is used as > >> > a reference counter. This allows to avoid accidental > >> > refcounter overflows that might lead to

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Alexey Dobriyan
On Mon, Jul 10, 2017 at 11:37 AM, Eric W. Biederman wrote: > "Reshetova, Elena" writes: > > 2>> Elena Reshetova writes: >>> >>> > refcount_t type and corresponding API should be >>> > used instead of atomic_t when the

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Alexey Dobriyan
On Mon, Jul 10, 2017 at 11:37 AM, Eric W. Biederman wrote: > "Reshetova, Elena" writes: > > 2>> Elena Reshetova writes: >>> >>> > refcount_t type and corresponding API should be >>> > used instead of atomic_t when the variable is used as >>> > a reference counter. This allows to avoid

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Eric W. Biederman
"Reshetova, Elena" writes: 2>> Elena Reshetova writes: >> >> > refcount_t type and corresponding API should be >> > used instead of atomic_t when the variable is used as >> > a reference counter. This allows to avoid accidental >> >

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Eric W. Biederman
"Reshetova, Elena" writes: 2>> Elena Reshetova writes: >> >> > refcount_t type and corresponding API should be >> > used instead of atomic_t when the variable is used as >> > a reference counter. This allows to avoid accidental >> > refcounter overflows that might lead to use-after-free >> >

RE: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Reshetova, Elena
> Elena Reshetova writes: > > > refcount_t type and corresponding API should be > > used instead of atomic_t when the variable is used as > > a reference counter. This allows to avoid accidental > > refcounter overflows that might lead to use-after-free > >

RE: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-10 Thread Reshetova, Elena
> Elena Reshetova writes: > > > refcount_t type and corresponding API should be > > used instead of atomic_t when the variable is used as > > a reference counter. This allows to avoid accidental > > refcounter overflows that might lead to use-after-free > > situations. > > In this patch you

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-09 Thread Eric W. Biederman
Elena Reshetova writes: > refcount_t type and corresponding API should be > used instead of atomic_t when the variable is used as > a reference counter. This allows to avoid accidental > refcounter overflows that might lead to use-after-free > situations. In this

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-09 Thread Eric W. Biederman
Elena Reshetova writes: > refcount_t type and corresponding API should be > used instead of atomic_t when the variable is used as > a reference counter. This allows to avoid accidental > refcounter overflows that might lead to use-after-free > situations. In this patch you can see all of the

[PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-07 Thread Elena Reshetova
refcount_t type and corresponding API should be used instead of atomic_t when the variable is used as a reference counter. This allows to avoid accidental refcounter overflows that might lead to use-after-free situations. Signed-off-by: Elena Reshetova Signed-off-by:

[PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-07-07 Thread Elena Reshetova
refcount_t type and corresponding API should be used instead of atomic_t when the variable is used as a reference counter. This allows to avoid accidental refcounter overflows that might lead to use-after-free situations. Signed-off-by: Elena Reshetova Signed-off-by: Hans Liljestrand

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-05-28 Thread Manfred Spraul
On 05/27/2017 09:41 PM, Kees Cook wrote: On Mon, Feb 20, 2017 at 3:29 AM, Elena Reshetova wrote: refcount_t type and corresponding API should be used instead of atomic_t when the variable is used as a reference counter. This allows to avoid accidental refcounter

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-05-28 Thread Manfred Spraul
On 05/27/2017 09:41 PM, Kees Cook wrote: On Mon, Feb 20, 2017 at 3:29 AM, Elena Reshetova wrote: refcount_t type and corresponding API should be used instead of atomic_t when the variable is used as a reference counter. This allows to avoid accidental refcounter overflows that might lead to

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-05-27 Thread Kees Cook
On Mon, Feb 20, 2017 at 3:29 AM, Elena Reshetova wrote: > refcount_t type and corresponding API should be > used instead of atomic_t when the variable is used as > a reference counter. This allows to avoid accidental > refcounter overflows that might lead to

Re: [PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-05-27 Thread Kees Cook
On Mon, Feb 20, 2017 at 3:29 AM, Elena Reshetova wrote: > refcount_t type and corresponding API should be > used instead of atomic_t when the variable is used as > a reference counter. This allows to avoid accidental > refcounter overflows that might lead to use-after-free > situations. > >

[PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-02-20 Thread Elena Reshetova
refcount_t type and corresponding API should be used instead of atomic_t when the variable is used as a reference counter. This allows to avoid accidental refcounter overflows that might lead to use-after-free situations. Signed-off-by: Elena Reshetova Signed-off-by:

[PATCH 1/3] ipc: convert ipc_namespace.count from atomic_t to refcount_t

2017-02-20 Thread Elena Reshetova
refcount_t type and corresponding API should be used instead of atomic_t when the variable is used as a reference counter. This allows to avoid accidental refcounter overflows that might lead to use-after-free situations. Signed-off-by: Elena Reshetova Signed-off-by: Hans Liljestrand