[Libvirt-cim] [PATCH 1 of 6] Turns out that the sdc_rasd_prop functions might need to report status or see the passed in reference. This updates the typedef and the memory functions, since those are already in the tree

Jay Gagnon grendel at linux.vnet.ibm.com
Fri Nov 9 15:21:37 UTC 2007


Kaitlin Rupert wrote:
> Jay Gagnon wrote:
>> # HG changeset patch
>> # User Jay Gagnon <grendel at linux.vnet.ibm.com>
>> # Date 1194557266 18000
>> # Node ID fd6deb234772b44fe549a3513a47115a01e20f7d
>> # Parent  bd1b1067d106ecc1546c3b2436a43f59f8eaba15
>> Turns out that the sdc_rasd_prop functions might need to report
>> status or see the passed in reference.  This updates the typedef and
>> the memory functions, since those are already in the tree.
>> Signed-off-by: Jay Gagnon <grendel at linux.vnet.ibm.com>
>>
>>
>>   
> Why is the ref param needed here?  Is that something you'll be adding
> later?
>
For the most part, the ref param is unneeded, but some of the callbacks
(disk_max for example) need to look at the ref to determine which Pool
they are being asked about.  All of the callbacks need to have the same
header because the sdc_rasd struct stores function pointers for the
callbacks and I really didn't want to do it as the always foreboding
(void *).

-- 

-Jay




More information about the Libvirt-cim mailing list