Re: [PATCH 1/2] Disable UV BAU by default

2016-03-23 Thread Alex Thorlton
On Wed, Mar 23, 2016 at 06:20:26PM +0100, Thomas Gleixner wrote: > On Wed, 23 Mar 2016, Alex Thorlton wrote: > > This was actually what I initially wrote, but we decided to go with the > > on/off switch instead, because, in the UV4 time-frame, we're hoping to > > get a few things changed so that

Re: [PATCH 1/2] Disable UV BAU by default

2016-03-23 Thread Alex Thorlton
On Wed, Mar 23, 2016 at 06:20:26PM +0100, Thomas Gleixner wrote: > On Wed, 23 Mar 2016, Alex Thorlton wrote: > > This was actually what I initially wrote, but we decided to go with the > > on/off switch instead, because, in the UV4 time-frame, we're hoping to > > get a few things changed so that

Re: [PATCH 1/2] Disable UV BAU by default

2016-03-23 Thread Thomas Gleixner
On Wed, 23 Mar 2016, Alex Thorlton wrote: > This was actually what I initially wrote, but we decided to go with the > on/off switch instead, because, in the UV4 time-frame, we're hoping to > get a few things changed so that we can default to having the bau *on* > for the new UV4 systems. > > I

Re: [PATCH 1/2] Disable UV BAU by default

2016-03-23 Thread Thomas Gleixner
On Wed, 23 Mar 2016, Alex Thorlton wrote: > This was actually what I initially wrote, but we decided to go with the > on/off switch instead, because, in the UV4 time-frame, we're hoping to > get a few things changed so that we can default to having the bau *on* > for the new UV4 systems. > > I

Re: [PATCH 1/2] Disable UV BAU by default

2016-03-23 Thread Alex Thorlton
On Wed, Mar 23, 2016 at 12:27:44PM +0100, Thomas Gleixner wrote: > On Mon, 21 Mar 2016, Alex Thorlton wrote: > > First of all, please use proper patch prefixes. > > x86/platform/uv: Ah - sorry about that! > And please fold the documentation change into the patch which changes the >

Re: [PATCH 1/2] Disable UV BAU by default

2016-03-23 Thread Alex Thorlton
On Wed, Mar 23, 2016 at 12:27:44PM +0100, Thomas Gleixner wrote: > On Mon, 21 Mar 2016, Alex Thorlton wrote: > > First of all, please use proper patch prefixes. > > x86/platform/uv: Ah - sorry about that! > And please fold the documentation change into the patch which changes the >

Re: [PATCH 1/2] Disable UV BAU by default

2016-03-23 Thread Thomas Gleixner
On Mon, 21 Mar 2016, Alex Thorlton wrote: First of all, please use proper patch prefixes. x86/platform/uv: And please fold the documentation change into the patch which changes the parameter. > static int timeout_us; > -static int nobau; > +static int nobau = 1; > static int nobau_perm;

Re: [PATCH 1/2] Disable UV BAU by default

2016-03-23 Thread Thomas Gleixner
On Mon, 21 Mar 2016, Alex Thorlton wrote: First of all, please use proper patch prefixes. x86/platform/uv: And please fold the documentation change into the patch which changes the parameter. > static int timeout_us; > -static int nobau; > +static int nobau = 1; > static int nobau_perm;

[PATCH 1/2] Disable UV BAU by default

2016-03-21 Thread Alex Thorlton
For several years, the common practice has been to boot UVs with the "nobau" parameter on the command line, to disable the BAU. We've decided that it makes more sense to just disable the BAU by default in the kernel, and provide the option to turn it on, if desired. Signed-off-by: Alex Thorlton

[PATCH 1/2] Disable UV BAU by default

2016-03-21 Thread Alex Thorlton
For several years, the common practice has been to boot UVs with the "nobau" parameter on the command line, to disable the BAU. We've decided that it makes more sense to just disable the BAU by default in the kernel, and provide the option to turn it on, if desired. Signed-off-by: Alex Thorlton