[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 0/3] Add Processor/MemoryDevice information to virSysinfoRead() API



Hi,

On Fri, 24 Jun 2011 22:38:44 +0800
Daniel Veillard <veillard redhat com> wrote:

> On Fri, Jun 24, 2011 at 03:20:29PM +0800, Daniel Veillard wrote:
> > On Thu, Jun 23, 2011 at 01:16:36PM +0900, Minoru Usui wrote:
> [...]
> >   My question is "do you need all those informations ???", my concern
> > is that already on my small desktop the output is 3KB and I assume on a
> > very large box we could hit hundreds of kilobytes.
> >   Also empty entries (i.e. zero lenght or spaces) should IMHO be
> > removed.
> > 
> >   I'm ready to commit those with the memory leak fixes, but a bit more
> > feedback would be welcome :-)
> 
>   I just commited and just seeing Dan comments now, I think we will
>   clean this up beginning of next week

I'm sorry for late reply.
Thank you for commiting my patch.

I'll refrect Dan's comment.
 
> Daniel
> 
> -- 
> Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
> daniel veillard com  | Rpmfind RPM search engine http://rpmfind.net/
> http://veillard.com/ | virtualization library  http://libvirt.org/


-- 
Minoru Usui <usui mxm nes nec co jp>


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]