WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-cim

[Xen-cim] [PATCH] Fix the References and ReferenceNames function in Xen_

To: <xen-cim@xxxxxxxxxxxxxxxxxxx>
Subject: [Xen-cim] [PATCH] Fix the References and ReferenceNames function in Xen_HostedComputerSystem
From: "Subrahmanian, Raj" <raj.subrahmanian@xxxxxxxxxx>
Date: Fri, 14 Jul 2006 16:13:52 -0400
Delivery-date: Fri, 14 Jul 2006 13:14:39 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-cim-request@lists.xensource.com?subject=help>
List-id: xen-cim mailing list <xen-cim.lists.xensource.com>
List-post: <mailto:xen-cim@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-cim>, <mailto:xen-cim-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-cim>, <mailto:xen-cim-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-cim-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acanch4+/2DJW1LLSluj6S1vPO6uOAADjwxQAABgTnA=
Thread-topic: [PATCH] Fix the References and ReferenceNames function in Xen_HostedComputerSystem
Resending patch as per Xen standards.
http://wiki.xensource.com/xenwiki/SubmittingXenPatches

Signed-off-by : Raj Subrahmanian <raj.subrahmanian@xxxxxxxxxx>

Raj

-----Original Message-----
From: Subrahmanian, Raj
Sent: Friday, July 14, 2006 1:36 PM
To: 'xen-cim@xxxxxxxxxxxxxxxxxxx'
Subject: RE: [Xen-cim] Patch for review

This patch fixes the 'not checking the returns from the CIMOM in
References and ReferenceNames' problem.
The key name checking is really not necessary to fix this problem. (As
an aside, I dont know if any key ties the Host ComputerSystem class and
the Virtual (Xen_ComputerSystem) class.) This patch fixes the
Xen_HostedComputerSystem file. It is quite trivial to do it for the
other classes, but I will wait till the SMASH issue is settled.

Raj

-----Original Message-----
From: xen-cim-bounces@xxxxxxxxxxxxxxxxxxx
[mailto:xen-cim-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Jim Fehlig
Sent: Wednesday, May 31, 2006 9:34 PM
To: Szymanski, Lukasz K
Cc: xen-cim@xxxxxxxxxxxxxxxxxxx
Subject: Re: [Xen-cim] Patch for review

Szymanski, Lukasz K wrote:

> Hello All -
>
> This patch adapts Raj's Xen_HostedComputerSystem patch to other
> Xen_Hosted* files:
>
> Xen_HostedDisk.mof, Xen_HostedDisk.c, Xen_HostedProcessor.mof, 
> Xen_HostedProcessor.mof, Xen_HostedMemory.mof, Xen_HostedMemory.c, 
> Xen_HostedNetworkPort.mof, Xen_HostedNetworkPort.c
>

The problem with this patch, and Raj's previous one apparently :-), is
that there is no filtering of object paths returned via upcalls to cimom
for instance names of the associated thing.  For example, in
ReferenceNames() in Xen_HostedMemory.c we make an upcall to get all
instances of CIM_Memory when it is the target class.  But Xen_Memory is
a subclass of CIM_Memory so you get instances of Xen_Memory (which is
the source class) and associate it with itself or other instances of
Xen_Memory for other domains.  If I list associations of Xen_Memory for
domain 1 using this patch, I get a Xen_HostedMemory association for each
Xen_Memory object produced by other domains.

Make sense?  I'm quite tired now so this may be a smoked explanation
:-).

Thanks,
Jim


_______________________________________________
Xen-cim mailing list
Xen-cim@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-cim

<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-cim] [PATCH] Fix the References and ReferenceNames function in Xen_HostedComputerSystem, Subrahmanian, Raj <=