[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 17/22] virtio-pmem: prototype
From: |
David Hildenbrand |
Subject: |
Re: [Qemu-devel] [PATCH v3 17/22] virtio-pmem: prototype |
Date: |
Mon, 24 Sep 2018 10:06:18 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.0 |
On 24/09/2018 07:45, David Gibson wrote:
> On Thu, Sep 20, 2018 at 12:32:38PM +0200, David Hildenbrand wrote:
>> From: Pankaj Gupta <address@hidden>
>>
>> This is the current protoype of virtio-pmem. Support will require
>> machine changes for the architectures that will support it, so it will
>> not yet be compiled.
>>
>> Signed-off-by: Pankaj Gupta <address@hidden>
>> [ MemoryDevice/MemoryRegion changes, cleanups, addr property "memaddr",
>> split up patches ]
>> Signed-off-by: David Hildenbrand <address@hidden>
>
> Reviewed-by: David Gibson <address@hidden>
>
> Seems generally sane. Is there a (craft?) virtio-pmem spec around to
> see what this is actually trying to implement?
I am not aware of any. The first goal is to get something implemented
that works and doesn't break important concepts. E.g. hotplugging memory
devices was one of these concepts that Igor saw as a potential problem
for getting virtio-pmem upstream.
But maybe Pankaj already has some draft version of a spec lying around.
>
> And one nit..
>
> [snip]
>> +static int worker_cb(void *opaque)
>> +{
>> + VirtIODeviceRequest *req = opaque;
>> + int err = 0;
>> +
>> + printf("\n performing flush ...");
>
> .. I assume the plan is to remove these debugging prints..
>
Yes, these are still in place for verifying that the machinery works.
--
Thanks,
David / dhildenb
- Re: [Qemu-devel] [PATCH v3 16/22] memory-device: add optional function get_device_id(), (continued)
- [Qemu-devel] [PATCH v3 17/22] virtio-pmem: prototype, David Hildenbrand, 2018/09/20
- Re: [Qemu-devel] [PATCH v3 17/22] virtio-pmem: prototype, Markus Armbruster, 2018/09/21
- Re: [Qemu-devel] [PATCH v3 17/22] virtio-pmem: prototype, David Hildenbrand, 2018/09/21
- Re: [Qemu-devel] [PATCH v3 17/22] virtio-pmem: prototype, Pankaj Gupta, 2018/09/21
- Re: [Qemu-devel] [PATCH v3 17/22] virtio-pmem: prototype, Markus Armbruster, 2018/09/21
- Re: [Qemu-devel] [PATCH v3 17/22] virtio-pmem: prototype, David Hildenbrand, 2018/09/21
- Re: [Qemu-devel] [PATCH v3 17/22] virtio-pmem: prototype, Markus Armbruster, 2018/09/21
- Re: [Qemu-devel] [PATCH v3 17/22] virtio-pmem: prototype, David Hildenbrand, 2018/09/21
Re: [Qemu-devel] [PATCH v3 17/22] virtio-pmem: prototype, David Gibson, 2018/09/24
[Qemu-devel] [PATCH v3 18/22] virtio-pci: proxy for virtio-pmem, David Hildenbrand, 2018/09/20
[Qemu-devel] [PATCH v3 19/22] hmp: handle virtio-pmem when printing memory device infos, David Hildenbrand, 2018/09/20
[Qemu-devel] [PATCH v3 20/22] numa: handle virtio-pmem in NUMA stats, David Hildenbrand, 2018/09/20
[Qemu-devel] [PATCH v3 21/22] virtio-pmem: hotplug support functions, David Hildenbrand, 2018/09/20