Messages in this thread | | | Date | Thu, 30 Jul 2015 11:01:24 -0700 (PDT) | From | Vikas Shivappa <> | Subject | Re: [PATCH 5/9] x86/intel_rdt: Add new cgroup and Class of service management |
| |
On Tue, 28 Jul 2015, Peter Zijlstra wrote:
> On Wed, Jul 01, 2015 at 03:21:06PM -0700, Vikas Shivappa wrote: >> +struct clos_cbm_map { >> + unsigned long cache_mask; >> + unsigned int clos_refcnt; >> +}; > > This structure is not a map at all, its the map value. Furthermore, > cache_mask seems a confusing name for the capacity bitmask (CBM).
clos_cbm_table ? since its really a table which is indexed by the clos.
will fix the mask names. Thanks, Vikas >
| |