[Libvirt-cim] [PATCH 1 of 2] Add get_rasd_instance_from_ref() to RASD

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Tue Nov 20 20:52:57 UTC 2007


Jay Gagnon wrote:
> Kaitlin Rupert wrote:
>   
>> # HG changeset patch
>> # User Kaitlin Rupert <karupert at us.ibm.com>
>> # Date 1195544369 28800
>> # Node ID 5bbff08351e4c05cb8ce63fe905f6742adc9b119
>> # Parent  a71690c7cf6caae2a3f55461980159b8d8fb92e3
>> Add get_rasd_instance_from_ref() to RASD.
>>
>> This allows other providers to get a specific RASD instance without having to do a CBGetInstance().
>>
>> Signed-off-by: Kaitlin Rupert <karupert at us.ibm.com>
>>
>> diff -r a71690c7cf6c -r 5bbff08351e4 src/Virt_RASD.c
>> --- a/src/Virt_RASD.c	Mon Nov 19 23:39:25 2007 -0800
>> +++ b/src/Virt_RASD.c	Mon Nov 19 23:39:29 2007 -0800
>> @@ -176,6 +176,7 @@ static CMPIInstance *rasd_from_vdev(cons
>>
>>  static CMPIInstance *get_rasd_instance(const CMPIContext *context,
>>                                         const CMPIObjectPath *ns,
>> +                                       const CMPIBroker *broker,
>>                                         const char *id,
>>                                         const uint16_t type)
>>  {
>> @@ -191,13 +192,13 @@ static CMPIInstance *get_rasd_instance(c
>>          if (!ret)
>>                  return NULL;
>>
>> -        conn = lv_connect(_BROKER, &s);
>> +        conn = lv_connect(broker, &s);
>>
>>   
>>     
> Should this part be connect_by_classname() now?  Other than that, looks
> great.
>
>
>   
Yes, thanks.  I think I didn't have the connect_by_classname() patch 
applied when I generated this.  I'll synch up my tree and resend.. =)

-- 
Kaitlin Rupert
IBM Linux Technology Center
karupert at us.ibm.com





More information about the Libvirt-cim mailing list