Re: [PATCH v2 1/8] locking: Fix compilation of __WW_MUTEX_INITIALIZER

2016-12-16 Thread Chris Wilson
On Fri, Dec 16, 2016 at 03:50:40PM +0100, Peter Zijlstra wrote: > > > Chris, did this series at all depend on the other ww_mutex patches? It is independent. My goal was to try and hit the bugs Nicolai reported. Most of it should be covered by locktorture if we add ww_mutex support to it. -Chris

Re: [PATCH v2 1/8] locking: Fix compilation of __WW_MUTEX_INITIALIZER

2016-12-16 Thread Chris Wilson
On Fri, Dec 16, 2016 at 03:50:40PM +0100, Peter Zijlstra wrote: > > > Chris, did this series at all depend on the other ww_mutex patches? It is independent. My goal was to try and hit the bugs Nicolai reported. Most of it should be covered by locktorture if we add ww_mutex support to it. -Chris

Re: [PATCH v2 1/8] locking: Fix compilation of __WW_MUTEX_INITIALIZER

2016-12-16 Thread Peter Zijlstra
Chris, did this series at all depend on the other ww_mutex patches?

Re: [PATCH v2 1/8] locking: Fix compilation of __WW_MUTEX_INITIALIZER

2016-12-16 Thread Peter Zijlstra
Chris, did this series at all depend on the other ww_mutex patches?

[PATCH v2 1/8] locking: Fix compilation of __WW_MUTEX_INITIALIZER

2016-12-01 Thread Chris Wilson
>From conflicting macro parameters, passing the wrong name to __MUTEX_INITIALIZER and a stray '\', #define __WW_MUTEX_INITIALIZER was very unhappy. One unnecessary change was to choose to pass _class instead of implicitly taking the address of the class within the macro. Fixes: 1b375dc30710

[PATCH v2 1/8] locking: Fix compilation of __WW_MUTEX_INITIALIZER

2016-12-01 Thread Chris Wilson
>From conflicting macro parameters, passing the wrong name to __MUTEX_INITIALIZER and a stray '\', #define __WW_MUTEX_INITIALIZER was very unhappy. One unnecessary change was to choose to pass _class instead of implicitly taking the address of the class within the macro. Fixes: 1b375dc30710