From: Mihai Carabas <mihai.carabas@oracle.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Yan Vugenfirer <yvugenfi@redhat.com>,
QEMU Developers <qemu-devel@nongnu.org>,
Gerd Hoffmann <kraxel@redhat.com>
Subject: Re: [PATCH v6] Add support for pvpanic pci device
Date: Wed, 27 Jan 2021 20:49:43 +0200 [thread overview]
Message-ID: <fc9ec737-6dea-c4c4-5a29-7cc50e92351e@oracle.com> (raw)
In-Reply-To: <CAFEAcA9SBX-sT983TsiRu9TSBWrZDWe=LM5cAJNxmHxa5ud0pw@mail.gmail.com>
La 27.01.2021 19:47, Peter Maydell a scris:
> On Wed, 27 Jan 2021 at 15:46, Mihai Carabas <mihai.carabas@oracle.com> wrote:
>> This patchset adds support for pvpanic pci device.
>>
>> v3:
>> - patch 1: made pvpanic isa device available only for PC, compile pvpanic-test
>> only when isa device is present
>> - patch 2: fixed device id to 0x0011, used OBJECT_DECLARE_TYPE,
>> PVPANIC_PCI_DEVICE, added VMSTATE_PCI_DEVICE, removed INTERFACE_PCIE_DEVICE
>> - patch 3: fixed documentation
>> - patch 4: add a qtest for pvpanic-pci
>>
>> v4:
>> - added Rb/Ack on patches
>> - modify test case to include -action parameter that was recently added and also
>> to be on par with the pvpanic ISA device testing
>>
>> v5:
>> - added subsystem_vendor_id and subsystem_id needed for MS WHQL tests
>>
>> v6:
>> - removed subsystem_vendor_id and subsystem_id as they are filed out by default
>> if empty
>> - do not compile pvpanic-pci-test for ppc64 as we our tests do not support that
>> platform
> Why doesn't the test work on PPC ?
I have to re-write the test suite in order to use a PCI bus specific to
pp64 platform. If we want to have test cases for ppc64, I can do this
with additional logic.
Thank you,
Mihai
next prev parent reply other threads:[~2021-01-27 18:51 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-27 14:59 [PATCH v6] Add support for pvpanic pci device Mihai Carabas
2021-01-27 14:59 ` [PATCH v6 1/4] hw/misc/pvpanic: split-out generic and bus dependent code Mihai Carabas
2021-01-27 14:59 ` [PATCH v6 2/4] hw/misc/pvpanic: add PCI interface support Mihai Carabas
2021-01-27 14:59 ` [PATCH v6 3/4] pvpanic : update pvpanic spec document Mihai Carabas
2021-01-27 14:59 ` [PATCH v6 4/4] tests/qtest: add a test case for pvpanic-pci Mihai Carabas
2021-01-27 17:47 ` [PATCH v6] Add support for pvpanic pci device Peter Maydell
2021-01-27 18:49 ` Mihai Carabas [this message]
2021-01-28 17:35 ` 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=fc9ec737-6dea-c4c4-5a29-7cc50e92351e@oracle.com \
--to=mihai.carabas@oracle.com \
--cc=kraxel@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=yvugenfi@redhat.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).