Re: [Qemu-block] [Qemu-devel] [PATCH v9 0/8] Optimize VMDK I/O by allocating multiple clusters

2017-11-03 Thread John Snow


On 11/02/2017 11:50 PM, Fam Zheng wrote:
> On Thu, 11/02 17:02, John Snow wrote:
>>
>>
>> On 10/20/2017 02:53 AM, Ashijeet Acharya wrote:
>>>
>>> On Fri, Oct 20, 2017 at 11:58 Fam Zheng >> > wrote:
>>>
>>> On Mon, 10/09 22:12, Fam Zheng wrote:
>>> > On Mon, 10/09 18:29, Ashijeet Acharya wrote:
>>> > > Optimization test results:
>>> > >
>>> > > This patch series improves 128 KB sequential write performance to an
>>> > > empty VMDK file by 54%
>>> > >
>>> > > Benchmark command: ./qemu-img bench -w -c 1024 -s 128K -d 1 -t
>>> none -f
>>> > > vmdk test.vmdk
>>> > >
>>> > > Changes in v9:
>>> > > - rebase the series
>>> >
>>> > Thanks, looks good to me, applied:
>>> >
>>> > https://github.com/famz/qemu/tree/staging
>>>
>>> Ashijeet: I've been testing my branch and it seems installing
>>> Fedora/CentOS to a
>>> VMDK image is broken with your patches applied. Both guest and QEMU are
>>> responsive, but the installing of packages stops to make any
>>> progress at some
>>> point:
>>>
>>> Installing rootfiles.noarch (317/318)
>>> Installing langpacks-en.noarch (318/318)
>>> Performing post-installation setup tasks
>>> Configuring fedora-release.noarch
>>> Configuring filesystem.x86_64
>>> Configuring GeoIP-GeoLite-data.noarch
>>> Configuring python3.x86_64
>>> Configuring fedora-logos.x86_64
>>> Configuring kernel-core.x86_64
>>>
>>> # hang here
>>>
>>> Can you reproduce this on your machine?
>>>
>>> My command line is something like this:
>>>
>>> qemu-system-x86_64 -enable-kvm -cpu host -m 1G -qmp
>>> unix:/home/fam/.q/qemu-8DOC9EF4/qmp,server,nowait -name 8DOC9EF4
>>> -netdev user,id=vnet,hostfwd=:0.0.0.0 :10022-:22
>>> -device virtio-net-pci,netdev=vnet -drive
>>> file=/var/tmp/test2.vmdk,if=none,id=drive-1,cache=none,aio=native
>>> -device virtio-blk-pci,drive=drive-1 -cdrom
>>> /stor/iso/CentOS-6.9-x86_64-minimal.iso -pidfile
>>> /home/fam/.q/qemu-8DOC9EF4/pid
>>>
>>> qemu.git master doesn't have this problem. So I'll drop this series
>>> from the
>>> pull request until it is resolved.
>>>
>>>
>>> Fam: Alright, I will look into this but I cannot give you a deadline
>>> unfortunately. I will try my best to resolve this as soon as though.
>>>
>>> Ashijeet
>>>
>>>
>>>
>>> Fam
>>>
>>
>> Do we need to temporarily roll this back for the 2.11 release if it
>> cannot be addressed in time?
> 
> Hi John, it's not merged yet, I dropped it from my pull request.
> 
> Fam
> 
OK thanks, just trying to get back up to speed and making sure things
that found their way into my "pay attention to this" folder are taken
care of.

:)

--js



Re: [Qemu-block] [Qemu-devel] [PATCH v9 0/8] Optimize VMDK I/O by allocating multiple clusters

2017-11-02 Thread John Snow


On 10/20/2017 02:53 AM, Ashijeet Acharya wrote:
> 
> On Fri, Oct 20, 2017 at 11:58 Fam Zheng  > wrote:
> 
> On Mon, 10/09 22:12, Fam Zheng wrote:
> > On Mon, 10/09 18:29, Ashijeet Acharya wrote:
> > > Optimization test results:
> > >
> > > This patch series improves 128 KB sequential write performance to an
> > > empty VMDK file by 54%
> > >
> > > Benchmark command: ./qemu-img bench -w -c 1024 -s 128K -d 1 -t
> none -f
> > > vmdk test.vmdk
> > >
> > > Changes in v9:
> > > - rebase the series
> >
> > Thanks, looks good to me, applied:
> >
> > https://github.com/famz/qemu/tree/staging
> 
> Ashijeet: I've been testing my branch and it seems installing
> Fedora/CentOS to a
> VMDK image is broken with your patches applied. Both guest and QEMU are
> responsive, but the installing of packages stops to make any
> progress at some
> point:
> 
> Installing rootfiles.noarch (317/318)
> Installing langpacks-en.noarch (318/318)
> Performing post-installation setup tasks
> Configuring fedora-release.noarch
> Configuring filesystem.x86_64
> Configuring GeoIP-GeoLite-data.noarch
> Configuring python3.x86_64
> Configuring fedora-logos.x86_64
> Configuring kernel-core.x86_64
> 
> # hang here
> 
> Can you reproduce this on your machine?
> 
> My command line is something like this:
> 
> qemu-system-x86_64 -enable-kvm -cpu host -m 1G -qmp
> unix:/home/fam/.q/qemu-8DOC9EF4/qmp,server,nowait -name 8DOC9EF4
> -netdev user,id=vnet,hostfwd=:0.0.0.0 :10022-:22
> -device virtio-net-pci,netdev=vnet -drive
> file=/var/tmp/test2.vmdk,if=none,id=drive-1,cache=none,aio=native
> -device virtio-blk-pci,drive=drive-1 -cdrom
> /stor/iso/CentOS-6.9-x86_64-minimal.iso -pidfile
> /home/fam/.q/qemu-8DOC9EF4/pid
> 
> qemu.git master doesn't have this problem. So I'll drop this series
> from the
> pull request until it is resolved.
> 
> 
> Fam: Alright, I will look into this but I cannot give you a deadline
> unfortunately. I will try my best to resolve this as soon as though.
> 
> Ashijeet
> 
> 
> 
> Fam
> 

Do we need to temporarily roll this back for the 2.11 release if it
cannot be addressed in time?



Re: [Qemu-block] [Qemu-devel] [PATCH v9 0/8] Optimize VMDK I/O by allocating multiple clusters

2017-10-20 Thread Ashijeet Acharya
On Fri, Oct 20, 2017 at 11:58 Fam Zheng  wrote:

> On Mon, 10/09 22:12, Fam Zheng wrote:
> > On Mon, 10/09 18:29, Ashijeet Acharya wrote:
> > > Optimization test results:
> > >
> > > This patch series improves 128 KB sequential write performance to an
> > > empty VMDK file by 54%
> > >
> > > Benchmark command: ./qemu-img bench -w -c 1024 -s 128K -d 1 -t none -f
> > > vmdk test.vmdk
> > >
> > > Changes in v9:
> > > - rebase the series
> >
> > Thanks, looks good to me, applied:
> >
> > https://github.com/famz/qemu/tree/staging
>
> Ashijeet: I've been testing my branch and it seems installing
> Fedora/CentOS to a
> VMDK image is broken with your patches applied. Both guest and QEMU are
> responsive, but the installing of packages stops to make any progress at
> some
> point:
>
> Installing rootfiles.noarch (317/318)
> Installing langpacks-en.noarch (318/318)
> Performing post-installation setup tasks
> Configuring fedora-release.noarch
> Configuring filesystem.x86_64
> Configuring GeoIP-GeoLite-data.noarch
> Configuring python3.x86_64
> Configuring fedora-logos.x86_64
> Configuring kernel-core.x86_64
>
> # hang here
>
> Can you reproduce this on your machine?
>
> My command line is something like this:
>
> qemu-system-x86_64 -enable-kvm -cpu host -m 1G -qmp
> unix:/home/fam/.q/qemu-8DOC9EF4/qmp,server,nowait -name 8DOC9EF4 -netdev
> user,id=vnet,hostfwd=:0.0.0.0:10022-:22 -device
> virtio-net-pci,netdev=vnet -drive
> file=/var/tmp/test2.vmdk,if=none,id=drive-1,cache=none,aio=native -device
> virtio-blk-pci,drive=drive-1 -cdrom /stor/iso/CentOS-6.9-x86_64-minimal.iso
> -pidfile /home/fam/.q/qemu-8DOC9EF4/pid
>
> qemu.git master doesn't have this problem. So I'll drop this series from
> the
> pull request until it is resolved.


Fam: Alright, I will look into this but I cannot give you a deadline
unfortunately. I will try my best to resolve this as soon as though.

Ashijeet

>
>
> Fam
>


Re: [Qemu-block] [Qemu-devel] [PATCH v9 0/8] Optimize VMDK I/O by allocating multiple clusters

2017-10-20 Thread Fam Zheng
On Mon, 10/09 22:12, Fam Zheng wrote:
> On Mon, 10/09 18:29, Ashijeet Acharya wrote:
> > Optimization test results:
> > 
> > This patch series improves 128 KB sequential write performance to an
> > empty VMDK file by 54%
> > 
> > Benchmark command: ./qemu-img bench -w -c 1024 -s 128K -d 1 -t none -f
> > vmdk test.vmdk
> > 
> > Changes in v9:
> > - rebase the series
> 
> Thanks, looks good to me, applied:
> 
> https://github.com/famz/qemu/tree/staging

Ashijeet: I've been testing my branch and it seems installing Fedora/CentOS to a
VMDK image is broken with your patches applied. Both guest and QEMU are
responsive, but the installing of packages stops to make any progress at some
point:

Installing rootfiles.noarch (317/318)
Installing langpacks-en.noarch (318/318)
Performing post-installation setup tasks
Configuring fedora-release.noarch
Configuring filesystem.x86_64
Configuring GeoIP-GeoLite-data.noarch
Configuring python3.x86_64
Configuring fedora-logos.x86_64
Configuring kernel-core.x86_64

# hang here

Can you reproduce this on your machine?

My command line is something like this:

qemu-system-x86_64 -enable-kvm -cpu host -m 1G -qmp 
unix:/home/fam/.q/qemu-8DOC9EF4/qmp,server,nowait -name 8DOC9EF4 -netdev 
user,id=vnet,hostfwd=:0.0.0.0:10022-:22 -device virtio-net-pci,netdev=vnet 
-drive file=/var/tmp/test2.vmdk,if=none,id=drive-1,cache=none,aio=native 
-device virtio-blk-pci,drive=drive-1 -cdrom 
/stor/iso/CentOS-6.9-x86_64-minimal.iso -pidfile /home/fam/.q/qemu-8DOC9EF4/pid

qemu.git master doesn't have this problem. So I'll drop this series from the
pull request until it is resolved.

Fam