Re: [PATCH 1/7] atomics/tty: add missing atomic_long_t * cast

2018-06-28 Thread Greg Kroah-Hartman
On Fri, Jun 22, 2018 at 05:55:59PM +0100, Mark Rutland wrote: > Hi, > > On Wed, Jun 06, 2018 at 04:00:42PM +0200, Greg Kroah-Hartman wrote: > > On Tue, Jun 05, 2018 at 04:53:34PM +0200, Peter Zijlstra wrote: > > > On Tue, Jun 05, 2018 at 04:43:04PM +0200, Peter Zijlstra wrote: > > > > > > > I can

Re: [PATCH 1/7] atomics/tty: add missing atomic_long_t * cast

2018-06-22 Thread Mark Rutland
Hi, On Wed, Jun 06, 2018 at 04:00:42PM +0200, Greg Kroah-Hartman wrote: > On Tue, Jun 05, 2018 at 04:53:34PM +0200, Peter Zijlstra wrote: > > On Tue, Jun 05, 2018 at 04:43:04PM +0200, Peter Zijlstra wrote: > > > > > I can make a proper patch, hold on. > > > > --- > > Subject: atomic/tty: Fix up

Re: [PATCH 1/7] atomics/tty: add missing atomic_long_t * cast

2018-06-06 Thread Greg Kroah-Hartman
On Tue, Jun 05, 2018 at 04:53:34PM +0200, Peter Zijlstra wrote: > On Tue, Jun 05, 2018 at 04:43:04PM +0200, Peter Zijlstra wrote: > > > I can make a proper patch, hold on. > > --- > Subject: atomic/tty: Fix up atomic abuse in ldsem > > Mark found ldsem_cmpxchg() needed an (atomic_long_t *) cast

Re: [PATCH 1/7] atomics/tty: add missing atomic_long_t * cast

2018-06-06 Thread Mark Rutland
On Tue, Jun 05, 2018 at 04:53:34PM +0200, Peter Zijlstra wrote: > On Tue, Jun 05, 2018 at 04:43:04PM +0200, Peter Zijlstra wrote: > > > I can make a proper patch, hold on. > > --- > Subject: atomic/tty: Fix up atomic abuse in ldsem > > Mark found ldsem_cmpxchg() needed an (atomic_long_t *) cast

Re: [PATCH 1/7] atomics/tty: add missing atomic_long_t * cast

2018-06-05 Thread Peter Zijlstra
On Tue, Jun 05, 2018 at 04:43:04PM +0200, Peter Zijlstra wrote: > I can make a proper patch, hold on. --- Subject: atomic/tty: Fix up atomic abuse in ldsem Mark found ldsem_cmpxchg() needed an (atomic_long_t *) cast to keep working after making the atomic_long interface type safe. Needing casts

Re: [PATCH 1/7] atomics/tty: add missing atomic_long_t * cast

2018-06-05 Thread Peter Zijlstra
On Tue, Jun 05, 2018 at 04:20:51PM +0200, Greg Kroah-Hartman wrote: > I think Peter Hurley was just trying to hid the atomic mess behind > function calls, which is why he didn't open-code it like you did here. No reason to then not use the right types though. I mean, you can still use the helper f

Re: [PATCH 1/7] atomics/tty: add missing atomic_long_t * cast

2018-06-05 Thread Greg Kroah-Hartman
On Tue, Jun 05, 2018 at 02:00:22PM +0200, Peter Zijlstra wrote: > On Tue, May 29, 2018 at 07:07:40PM +0100, Mark Rutland wrote: > > In ldsem_cmpxchg a pointer to unsigned long is passed to > > atomic_long_cmpxchg(), which expects a pointer to atomic_long_t. > > > - long tmp = atomic_long_cmpxchg

Re: [PATCH 1/7] atomics/tty: add missing atomic_long_t * cast

2018-06-05 Thread Peter Zijlstra
On Tue, May 29, 2018 at 07:07:40PM +0100, Mark Rutland wrote: > In ldsem_cmpxchg a pointer to unsigned long is passed to > atomic_long_cmpxchg(), which expects a pointer to atomic_long_t. > - long tmp = atomic_long_cmpxchg(&sem->count, *old, new); > + long tmp = atomic_long_cmpxchg((atomic

Re: [PATCH 1/7] atomics/tty: add missing atomic_long_t * cast

2018-05-29 Thread Greg Kroah-Hartman
On Tue, May 29, 2018 at 07:07:40PM +0100, Mark Rutland wrote: > In ldsem_cmpxchg a pointer to unsigned long is passed to > atomic_long_cmpxchg(), which expects a pointer to atomic_long_t. > > In preparation for making the atomic_long_* APIs type safe, add a cast > before passing the value to atomi

[PATCH 1/7] atomics/tty: add missing atomic_long_t * cast

2018-05-29 Thread Mark Rutland
In ldsem_cmpxchg a pointer to unsigned long is passed to atomic_long_cmpxchg(), which expects a pointer to atomic_long_t. In preparation for making the atomic_long_* APIs type safe, add a cast before passing the value to atomic_long_cmpxchg(). Similar is already done in ldsem_atomic_update() when