Messages in this thread | | | Date | Wed, 18 Jan 2017 10:05:15 +0100 (CET) | From | Thomas Gleixner <> | Subject | Re: [PATCH 3/8] x86/intel_rdt/mba: Improvements to handle more RDT resources like MBA |
| |
On Tue, 17 Jan 2017, Shivappa Vikas wrote: > On Mon, 16 Jan 2017, Thomas Gleixner wrote: > > > On Tue, 10 Jan 2017, Vikas Shivappa wrote: > > > This patch does some changes to get ready to handle more resources like > > > Memory b/w allocation(MBA). > > > > > > -Update the control registers only when user changes the controls(cbm for > > > Cache resources and Mem b/w for memory). Hence not sending IPIs on all > > > domains when user updates the control vals. > > > -Introduce next_enabled_resource rather than looping through all > > > resources while parsing each schemata line. The order of resources > > > should be anyways the same as the root schemata. > > > -Return error as soon as we detect a resource not entering all domain > > > values in schemata rather than waiting till we parse all resources. > > > > That looks all like reasonable optimizations and I have a hard time to > > understand why this is a prerequisite for the bandwidth support. > > > > And each of these changes is independent so they should be in seperate > > patches. > > Ok will split them. Its not a pre requisite for MBA. Should i send as a > seperate series ?
Yes please. That way we can apply preparatory things first and then concentrate on the new MBA functionality.
Thanks,
tglx
| |