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-devel

Re: [Xen-devel] [PATCH 1 of 5] Support getting topology info in libxl

To: Gianni Tedesco <gianni.tedesco@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH 1 of 5] Support getting topology info in libxl
From: Juergen Gross <juergen.gross@xxxxxxxxxxxxxx>
Date: Thu, 09 Dec 2010 14:19:44 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Delivery-date: Thu, 09 Dec 2010 05:20:47 -0800
Dkim-signature: v=1; a=rsa-sha256; c=simple/simple; d=ts.fujitsu.com; i=juergen.gross@xxxxxxxxxxxxxx; q=dns/txt; s=s1536b; t=1291900787; x=1323436787; h=message-id:date:from:mime-version:to:cc:subject: references:in-reply-to:content-transfer-encoding; z=Message-ID:=20<4D00D770.1080405@xxxxxxxxxxxxxx>|Date:=20 Thu,=2009=20Dec=202010=2014:19:44=20+0100|From:=20Juergen =20Gross=20<juergen.gross@xxxxxxxxxxxxxx>|MIME-Version: =201.0|To:=20Gianni=20Tedesco=20<gianni.tedesco@xxxxxxxxx m>|CC:=20"xen-devel@xxxxxxxxxxxxxxxxxxx"=20<xen-devel@lis ts.xensource.com>,=20=0D=0A=20Ian=20Jackson=20<Ian.Jackso n@xxxxxxxxxxxxx>|Subject:=20Re:=20[Xen-devel]=20[PATCH=20 1=20of=205]=20Support=20getting=20topology=20info=20in=20 libxl|References:=20<a1eb18bec4690b6687bc.1291898377@neha lem1>=20<1291900308.20366.1.camel@xxxxxxxxxxxxxxxxxxxxxx> |In-Reply-To:=20<1291900308.20366.1.camel@xxxxxxxxxxxxxxx rce.com>|Content-Transfer-Encoding:=207bit; bh=EFHFLyBxlXjnOcMo+NcEAypAYnwVegiat0uH/14ltGw=; b=XJaXvq7M022qHgEiiPJvIO9b3K0639Ngp8dkiLJqVb+T0vbWI9I18iq3 BxFJLoghhOfGUgKD1o7k8pUflyGxI/bN25qhs+apEgLAF/L2vZKNwITHC EEnF5/AIVpkMEQR1L68lJt3hc4mkiuP9em9JIBRqzZLt98HLSnN5Kn4wx DqOte7lQJTL9iKUKnRXG+AyLpQ4RO93pagQuRuqMidNb/jXQKkrTpsKQ1 DGH9cG785jLJ7JgfsvmYbIhXb9mdA;
Domainkey-signature: s=s1536a; d=ts.fujitsu.com; c=nofws; q=dns; h=X-SBRSScore:X-IronPort-AV:Received:X-IronPort-AV: Received:Received:Message-ID:Date:From:Organization: User-Agent:MIME-Version:To:CC:Subject:References: In-Reply-To:Content-Type:Content-Transfer-Encoding; b=bn7PTyccXxPo2TPRjI7WPb5JCTmQ4Fo4comdSppfgJsijR0lMlhelWTU BHG1tjMMIDgGK6HLyyv+Tl6QAVs8LKoPfWcO8syGiPuqkkYXz4pzB0SNS wR9bjglywtuhQq7XYCowTVs/wojDANJ4EHMo0SwwcC2zGQeBvPmUA+CCW E9N7HBa8+XAFK6DdFjigcezbd+6A7Ud8pTlOsBVIFGWRs0jLPgqNwt/7A ijvHYxRSzcjaLmYMFeYuIr/BDvnTC;
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1291900308.20366.1.camel@xxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: Fujitsu Technology Solutions
References: <a1eb18bec4690b6687bc.1291898377@nehalem1> <1291900308.20366.1.camel@xxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.15) Gecko/20101030 Iceowl/1.0b1 Icedove/3.0.10
Ian,

could you take my previous version of this patch (1 of 5), or would you like
me to send another complete patch series?


Juergen

On 12/09/10 14:11, Gianni Tedesco wrote:
On Thu, 2010-12-09 at 12:39 +0000, Juergen Gross wrote:

+PyObject *attrib__libxl_cpuarray_get(libxl_cpuarray *pptr)
+{
+    PyObject *list = NULL;
+    int i;
+
+    list = PyList_New(0);
+    for (i = 0; i<  pptr->entries; i++) {
+        if (pptr->array[i] == LIBXL_CPUARRAY_INVALID_ENTRY) {
+            PyList_Append(list, Py_None);
+            Py_INCREF(Py_None);


Actually I think I am the one who is confused. It is the other way
around. I need to fix other places that leak references due to this.

IOW, this was good to go first time around, tried to respond earlier but
my mail client barfed the message.

Sorry for the noise! :(

--
Juergen Gross                 Principal Developer Operating Systems
TSP ES&S SWE OS6                       Telephone: +49 (0) 89 3222 2967
Fujitsu Technology Solutions              e-mail: juergen.gross@xxxxxxxxxxxxxx
Domagkstr. 28                           Internet: ts.fujitsu.com
D-80807 Muenchen                 Company details: ts.fujitsu.com/imprint.html

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

<Prev in Thread] Current Thread [Next in Thread>