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

[Juergen Gross <juergen.gross@xxxxxxxxxxxxxx>]

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