[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC] virtio: Report new guest memory statistics pertin
From: |
Jamie Lokier |
Subject: |
Re: [Qemu-devel] [RFC] virtio: Report new guest memory statistics pertinent to memory ballooning (V2) |
Date: |
Mon, 9 Nov 2009 19:00:01 +0000 |
User-agent: |
Mutt/1.5.13 (2006-08-11) |
Adam Litke wrote:
> + s->stats.pswapin = has_feature(dev, VIRTIO_BALLOON_F_RPT_SWAP_OUT) ?
> + dev->stats.pswapin : -1;
(etc.)
Why not simply have the guest fill in the unused fields with -1, and
say that's how "no meaningful value" is represented in the ABI?
All guests have to know about all those fields anyway, for the
structure layout. Is there any benefit to specifying feature bits in
advance over simply storing -1 there?
-- Jamie
- Re: [Qemu-devel] Re: virtio: Add memory statistics reporting to the balloon driver, (continued)
- Re: [Qemu-devel] Re: virtio: Add memory statistics reporting to the balloon driver, Rusty Russell, 2009/11/10
- [Qemu-devel] Re: virtio: Add memory statistics reporting to the balloon driver, Anthony Liguori, 2009/11/10
- [Qemu-devel] Re: virtio: Add memory statistics reporting to the balloon driver, Rusty Russell, 2009/11/10
- [Qemu-devel] Re: virtio: Add memory statistics reporting to the balloon driver, Anthony Liguori, 2009/11/10
- [Qemu-devel] Re: virtio: Add memory statistics reporting to the balloon driver, Rusty Russell, 2009/11/10
- [Qemu-devel] Re: virtio: Add memory statistics reporting to the balloon driver, Adam Litke, 2009/11/11
- [Qemu-devel] Re: virtio: Add memory statistics reporting to the balloon driver, Rusty Russell, 2009/11/11
Re: [Qemu-devel] [RFC] virtio: Report new guest memory statistics pertinent to memory ballooning (V2),
Jamie Lokier <=
Re: [Qemu-devel] [RFC] virtio: Report new guest memory statistics pertinent to memory ballooning (V2), Jamie Lokier, 2009/11/09