Re: [libvirt] [PATCHv2] virsh: fix range of memtune command

2010-10-26 Thread Eric Blake
On 10/25/2010 11:06 PM, Nikunj A. Dadhania wrote: On Wed, 20 Oct 2010 13:29:45 -0600, Eric Blakeebl...@redhat.com wrote: * tools/virsh.c (cmdMemtune): Use long long for memory sizes. Simplify allocation, and plug memory leak. ACK Thanks; pushed. -- Eric Blake ebl...@redhat.com

Re: [libvirt] [PATCHv2] virsh: fix range of memtune command

2010-10-25 Thread Stefan Berger
On 10/20/2010 03:29 PM, Eric Blake wrote: * tools/virsh.c (cmdMemtune): Use long long for memory sizes. Simplify allocation, and plug memory leak. --- No need to cripple virsh with a 32-bit limit. Change from v1: fix compilation failure, rebase on top of other recent memtune fixes

Re: [libvirt] [PATCHv2] virsh: fix range of memtune command

2010-10-25 Thread Nikunj A. Dadhania
On Wed, 20 Oct 2010 13:29:45 -0600, Eric Blake ebl...@redhat.com wrote: * tools/virsh.c (cmdMemtune): Use long long for memory sizes. Simplify allocation, and plug memory leak. --- No need to cripple virsh with a 32-bit limit. Change from v1: fix compilation failure, rebase on top of