Re: [PATCH 12/17] imsm: create mdinfo list of disks in a container from supertype

[Dan Williams <dan.j.williams@intel.com>]

On 10/29/2010 7:24 AM, Czarnowska, Anna wrote:
>  From 882ad1e86e09304ce244d063e64fa1cc65fcc799 Mon Sep 17 00:00:00 2001
> From: Anna Czarnowska<anna.czarnowska@intel.com>
> Date: Wed, 27 Oct 2010 12:00:28 +0200
> Subject: [PATCH 12/17] imsm: create mdinfo list of disks in a container from supertype
>
> If getinfo_super is called on a container supertype we only get information
> on first disk. As a parameter it uses reference to preallocated
> mdinfo structure. Amending getinfo_super to return full list of disks
> would require ammending all previous calls and subsequently freeing memory
> allocated for mdinfo list.
> Function container_content that returns a mdinfo list is written
> specifically for assembly, performing actions not needed to just fill
> mdinfo. It also does not include spares so is unsuitable.
> As an alternative a new function getinfo_super_disks is created
> to obtain information about all disks states in array.
> Existing function sysfs_free is used to free memory
> allocated by getinfo_super_disks.
>
> Signed-off-by: Anna Czarnowska<anna.czarnowska@intel.com>
> Signed-off-by: Marcin Labun<marcin.labun@intel.com>

Yes, this is needed, but is an example of something that can hopefully 
go away after Neil's rework of the data structures returned from a load 
operation.  We shouldn't need a custom routine to retrieve this list. 
It should be available directly from the supertype.