From: Laurent Vivier <lvivier@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Andrew Jones <drjones@redhat.com>,
Alexander Gordeev <agordeev@redhat.com>,
QEMU Developers <qemu-devel@nongnu.org>,
"Michael S. Tsirkin" <mst@redhat.com>
Subject: Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space
Date: Fri, 16 Oct 2015 11:59:35 +0200 [thread overview]
Message-ID: <5620CA87.7060204@redhat.com> (raw)
In-Reply-To: <CAFEAcA8__FCC-4ZEvutBcdnZ5HbSe6RmVVieqd-BS2jnJPyH8w@mail.gmail.com>
On 16/10/2015 11:32, Peter Maydell wrote:
> On 16 October 2015 at 09:52, Laurent Vivier <lvivier@redhat.com> wrote:
>> I'm going to remove the zero address checking and try to start a qemu
>> for each of them to see which ones are broken.
>
> You need to also make sure there's a PCI card in there
> and that the guest maps it with a BAR at address zero
> (testing both MMIO BARs and IO BARs), and then exercise
> the guest sufficiently to check that behaviour of whatever
> it might be overlapping is still OK.
I think it will be hard to trigger this case. :(
For instance, on pseries, the card is put at address 0 only when it is
hotplugged, and on reboot it is moved to another address.
> You might find it easier to do by code inspection in
> some cases (boards which implement PCI as a simple
> "memory window into the PCI space" rather than with
> x86-style "anything in the background not covered by
> another device is PCI space" should be ok).
OK, I will...
Laurent
next prev parent reply other threads:[~2015-10-16 9:59 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-12 20:55 [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space Alexander Gordeev
2015-10-12 21:03 ` Peter Maydell
2015-10-13 6:31 ` Alexander Gordeev
2015-10-13 8:16 ` Peter Maydell
2015-10-13 12:48 ` Alexander Gordeev
2015-10-13 12:33 ` Peter Maydell
2015-10-13 12:47 ` Laurent Vivier
2015-10-13 13:04 ` Michael S. Tsirkin
2015-10-13 13:12 ` Peter Maydell
2015-10-13 13:19 ` Michael S. Tsirkin
2015-10-13 13:25 ` Peter Maydell
2015-10-13 13:55 ` Peter Maydell
2015-10-16 8:52 ` Laurent Vivier
2015-10-16 9:13 ` Michael S. Tsirkin
2015-10-16 9:32 ` Peter Maydell
2015-10-16 9:59 ` Laurent Vivier [this message]
2015-10-13 14:35 ` [Qemu-devel] [PATCH v2] " Alexander Gordeev
2015-10-13 17:36 ` Peter Maydell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5620CA87.7060204@redhat.com \
--to=lvivier@redhat.com \
--cc=agordeev@redhat.com \
--cc=drjones@redhat.com \
--cc=mst@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.