Re: [PATCH v5 05/16] qemu: Build command line for virtio-mem

2021-09-21 Thread David Hildenbrand
On 21.09.21 10:56, Michal Prívozník wrote: On 9/17/21 2:54 PM, David Hildenbrand wrote: On 15.09.21 13:07, Michal Prívozník wrote: On 9/14/21 3:05 PM, David Hildenbrand wrote: On 13.09.21 16:52, Michal Privoznik wrote: Nothing special is happening here. All important changes were done when

Re: [PATCH v5 05/16] qemu: Build command line for virtio-mem

2021-09-21 Thread Michal Prívozník
On 9/17/21 2:54 PM, David Hildenbrand wrote: > On 15.09.21 13:07, Michal Prívozník wrote: >> On 9/14/21 3:05 PM, David Hildenbrand wrote: >>> On 13.09.21 16:52, Michal Privoznik wrote: Nothing special is happening here. All important changes were done when for 'virtio-pmem' (adjusting

Re: [PATCH v5 05/16] qemu: Build command line for virtio-mem

2021-09-17 Thread David Hildenbrand
On 15.09.21 13:07, Michal Prívozník wrote: On 9/14/21 3:05 PM, David Hildenbrand wrote: On 13.09.21 16:52, Michal Privoznik wrote: Nothing special is happening here. All important changes were done when for 'virtio-pmem' (adjusting the code to put virtio memory on PCI bus, generating alias

Re: [PATCH v5 05/16] qemu: Build command line for virtio-mem

2021-09-15 Thread Michal Prívozník
On 9/15/21 1:26 PM, Jing Qi wrote: > Hi Michal, > The domain xml is attached. Thank you. I was able to reproduce but the problem is not in these patches. I can reproduce even without them. I have a fix that I will post shortly. Michal

Re: [PATCH v5 05/16] qemu: Build command line for virtio-mem

2021-09-15 Thread Jing Qi
Hi Michal, The domain xml is attached. Regards, Jing Qi On Wed, Sep 15, 2021 at 7:14 PM Michal Prívozník wrote: > On 9/14/21 3:05 PM, David Hildenbrand wrote: > > On 13.09.21 16:52, Michal Privoznik wrote: > >> Nothing special is happening here. All important changes were > >> done when for

Re: [PATCH v5 05/16] qemu: Build command line for virtio-mem

2021-09-15 Thread Michal Prívozník
On 9/15/21 8:16 AM, Jing Qi wrote: > Hi Michal, > I tried to test the virtio-mem with upstream version v7.7.0-136-g9b49c2c6d3 > adding the current patch > (with qemu-6.1.0-7.fc36.x86_64) - Hey, thanks for that! > > > 8388608 > 1179648 > 1179648 > > > > > > qemu64 >

Re: [PATCH v5 05/16] qemu: Build command line for virtio-mem

2021-09-15 Thread Michal Prívozník
On 9/14/21 3:05 PM, David Hildenbrand wrote: > On 13.09.21 16:52, Michal Privoznik wrote: >> Nothing special is happening here. All important changes were >> done when for 'virtio-pmem' (adjusting the code to put virtio >> memory on PCI bus, generating alias using >> qemuDomainDeviceAliasIndex().

Re: [PATCH v5 05/16] qemu: Build command line for virtio-mem

2021-09-15 Thread Jing Qi
Hi Michal, I tried to test the virtio-mem with upstream version v7.7.0-136-g9b49c2c6d3 adding the current patch (with qemu-6.1.0-7.fc36.x86_64) - 8388608 1179648 1179648 qemu64 ... 0 2048 131072

Re: [PATCH v5 05/16] qemu: Build command line for virtio-mem

2021-09-14 Thread David Hildenbrand
On 13.09.21 16:52, Michal Privoznik wrote: Nothing special is happening here. All important changes were done when for 'virtio-pmem' (adjusting the code to put virtio memory on PCI bus, generating alias using qemuDomainDeviceAliasIndex(). The only bit that might look suspicious is no prealloc

[PATCH v5 05/16] qemu: Build command line for virtio-mem

2021-09-13 Thread Michal Privoznik
Nothing special is happening here. All important changes were done when for 'virtio-pmem' (adjusting the code to put virtio memory on PCI bus, generating alias using qemuDomainDeviceAliasIndex(). The only bit that might look suspicious is no prealloc for virtio-mem. But if you think about it, the