How about for l3:
<control min="2816" avail=“56320” cbm_len=“20” scope=‘both’ reserved=“2816"/>

Well, yes, kind of what you had in your patches. Wasn't it without the
'cbm_len' and 'avail'? The 'cbm_len' is avail/min, so it's redundant
and avail is the same as the size of the whole cache, right? Also
'reserved' should not be there as that would have to be refreshed every
time the info is gathered and that's not what capabilities are for.
Also, if we say 'unified' instead of 'both', it sounds little more
consistent.

So basically, I'm thinking we were somewhere along the lines of:

<control min='2816' unit='B' scope=‘unified’/>

Or do I remember it wrong?
oh yeah, right!

for scope, it’s okay to use 'unified' to instead of ‘both’
for CDP enabled case would it be ?

1) 
 <control min='2816' unit='B' scope=‘data’/>
 <control min='2816' unit='B' scope=‘instruction’/>

or 

2) 

<control min='2816' unit='B' scope=‘data+instruction’/>

or 
3)

<control min='2816' unit='B' scope=‘cdp’/>

?

A correction, that would be <control min=‘2816 * 1024' unit='B' scope=‘unified’/>

the unit B is kinds of small for l3 cache.

Thx Eli.