[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init |
Date: |
Mon, 8 Aug 2011 16:14:27 +0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Aug 08, 2011 at 08:02:08AM -0500, Anthony Liguori wrote:
> On 08/08/2011 07:56 AM, Michael S. Tsirkin wrote:
> >On Mon, Aug 08, 2011 at 07:45:19AM -0500, Anthony Liguori wrote:
> >>On 08/08/2011 05:36 AM, Michael S. Tsirkin wrote:
> >>>>Thinking more closely, I don't think this right.
> >>>>
> >>>>Updating on map ensured that the config was refreshed after each
> >>>>time the bar was mapped. In the very least, the config needs to be
> >>>>refreshed during reset because the guest may write to the guest
> >>>>space which should get cleared after reset.
> >>>>
> >>>>Regards,
> >>>>
> >>>>Anthony Liguori
> >>>
> >>>Not sure I understand. Which register, for example,
> >>>do you have in mind?
> >>>Could you clarify please?
> >>
> >>Actually, you never need to call config_get() AFAICT. It's called
> >>in every read/write access.
> >
> >Every read, yes. But every write? Are you sure?
>
> Yeah, not on write, but I think this is a bug. get_config() should
> be called before doing the memcpy() in order to have a proper RMW.
>
> Regards,
>
> Anthony Liguori
Probably not noticeable because guests don't do the RMW
in practice.
We also send the config over on migration.
That's probably a bug as well ...
--
MST
- Re: [Qemu-devel] [PATCH v3 02/39] pci: add API to get a BAR's mapped address, (continued)
- [Qemu-devel] [PATCH v3 27/39] i6300esb: convert to memory API, Avi Kivity, 2011/08/04
- [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init, Avi Kivity, 2011/08/04
- Re: [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init, Anthony Liguori, 2011/08/05
- Re: [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init, Avi Kivity, 2011/08/07
- Re: [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init, Michael S. Tsirkin, 2011/08/08
- Re: [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init, Anthony Liguori, 2011/08/08
- Re: [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init, Avi Kivity, 2011/08/08
- Re: [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init, Michael S. Tsirkin, 2011/08/08
- Re: [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init, Anthony Liguori, 2011/08/08
- Re: [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init, Anthony Liguori, 2011/08/08
[Qemu-devel] [PATCH v3 21/39] ahci: convert to memory API, Avi Kivity, 2011/08/04
[Qemu-devel] [PATCH v3 03/39] vmsvga: don't remember pci BAR address in callback any more, Avi Kivity, 2011/08/04
[Qemu-devel] [PATCH v3 06/39] cirrus: simplify bitblt BAR access functions, Avi Kivity, 2011/08/04
[Qemu-devel] [PATCH v3 07/39] cirrus: simplify vga window mmio access functions, Avi Kivity, 2011/08/04
[Qemu-devel] [PATCH v3 19/39] ivshmem: convert to memory API, Avi Kivity, 2011/08/04
[Qemu-devel] [PATCH v3 22/39] intel-hda: convert to memory API, Avi Kivity, 2011/08/04