Re: [Powerpc-utils-devel] [PATCH v2] lsslot: Enhance memory output

[Paul C. <pc...@us...>]

On 05/15/2015 09:00 AM, Nathan Fontenot wrote:
> On 05/14/2015 02:33 PM, Paul Clarke wrote:
>> On 05/14/2015 11:31 AM, Nathan Fontenot wrote:
>>> This patch adds additional information to the output produced
>>> by the lsslot command for dynamic reconfiguration memory. The
>>> default output with this patch will add the address, drc index,
>>> and associativity for each LMB.
>>>
>>> #> lsslot -c mem
>>> Dynamic Reconfiguration Memory (LMB size 0x10000000)
>>> LMB2:
>>>       DRC Index: 80000001        Address: 10000000
>>>       Removable: Yes             Associativity: (index: 1) 0 1 2 2
>>>       Section(s): 1
>>> -- snip --
>>>
>>> Additionally, this patch allows users to specify the -d 4 option
>>> to lsslot for memory which will print information about all possible
>>> LMBs, not just the LMBs owned by the system.
>>
>> Is this supposed to read "print information about all LMBs installed in the system, not just the LMBs visible to the operating system" ?
>
> I think we could go with either wording. The number of possible LMBs is
> the max number of LMBs that can be possibly dlpar added to the lpar.
> When a partition is booted the device tree has information about all
> of these possible LMBs, even if they are not currently assigned to the
> partition.
>
> The wording I chose mirrors how I think of them, but am open to change
> if you think your wording is a better explanation.

To me, of course, my wording is clearer.  :-)  I was trying to clarify 
what "all possible LMBs" meant, and that "LMBs owned by the system" 
really wasn't for "the system" as a whole, but just for the operating 
system instance in which the command is executing.  (I hope I'm actually 
getting those respective meanings correct.)

PC