From: Ani Sinha <ani@anisinha.ca>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Ani Sinha <ani@anisinha.ca>, Igor Mammedov <imammedo@redhat.com>,
qemu-devel@nongnu.org
Subject: Re: [PATCH v3 0/3] tests/acpi/pcihp: add unit tests for hotplug on multifunction bridges for q35
Date: Thu, 21 Oct 2021 12:30:40 +0530 (IST) [thread overview]
Message-ID: <alpine.DEB.2.22.394.2110211228240.235596@anisinha-lenovo> (raw)
In-Reply-To: <20211021021635-mutt-send-email-mst@kernel.org>
On Thu, 21 Oct 2021, Michael S. Tsirkin wrote:
> On Thu, Oct 21, 2021 at 07:18:43AM +0530, Ani Sinha wrote:
> >
> >
> > On Wed, Oct 20, 2021 at 2:09 PM Michael S. Tsirkin <mst@redhat.com> wrote:
> >
> > On Thu, Oct 07, 2021 at 07:27:47PM +0530, Ani Sinha wrote:
> > > changelist:
> > > v3: removed "nodefaults" from the command line and rebased the patchset.
> > > v2: incorporated some of the feedbacks from Igor.
> > > v1 : initial RFC patch.
> >
> > This seems to break on s390 hosts for people. Likely an
> > endian-ness bug somewhere. Dropped for now - care tracking that down
> > and fixing so I can pick up the test again?
> >
> > Thanks!
> >
> >
> > So I take it this patch wasn't causing the issue since this has been merged to
> > master now?
> >
> >
>
> Yes, we knew the bug is in the tested functionality not the test of
> course. With help from Thomas I was able to fix the original bug.
> The fix with more detail is here:
>
> commit 0e464f7d993113119f0fd17b890831440734ce15
> Author: Michael S. Tsirkin <mst@redhat.com>
> Date: Wed Oct 20 05:48:54 2021 -0400
>
> pci: fix PCI resource reserve capability on BE
BUG is quite old, since the time the function was actually written -
70e1ee59bb94 ("hw/pci: introduce bridge-only vendor-specific capability to
provide some hints to firmware")
I doubt in BE systems it ever worked.
next prev parent reply other threads:[~2021-10-21 7:04 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-07 13:57 [PATCH v3 0/3] tests/acpi/pcihp: add unit tests for hotplug on multifunction bridges for q35 Ani Sinha
2021-10-07 13:57 ` [PATCH v3 1/3] tests/acpi/bios-tables-test: add and allow changes to a new q35 DSDT table blob Ani Sinha
2021-10-11 13:54 ` Igor Mammedov
2021-10-07 13:57 ` [PATCH v3 2/3] tests/acpi/pcihp: add unit tests for hotplug on multifunction bridges for q35 Ani Sinha
2021-10-11 13:53 ` Igor Mammedov
2021-10-07 13:57 ` [PATCH v3 3/3] tests/acpi/bios-tables-test: update DSDT blob for multifunction bridge test Ani Sinha
2021-10-11 13:54 ` Igor Mammedov
2021-10-20 8:39 ` [PATCH v3 0/3] tests/acpi/pcihp: add unit tests for hotplug on multifunction bridges for q35 Michael S. Tsirkin
2021-10-21 1:48 ` Ani Sinha
2021-10-21 6:19 ` Michael S. Tsirkin
2021-10-21 6:44 ` Ani Sinha
2021-10-21 7:00 ` Ani Sinha [this message]
-- strict thread matches above, loose matches on Subject: below --
2021-09-20 7:00 Ani Sinha
2021-09-23 7:14 ` Ani Sinha
2021-10-04 11:35 ` Ani Sinha
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=alpine.DEB.2.22.394.2110211228240.235596@anisinha-lenovo \
--to=ani@anisinha.ca \
--cc=imammedo@redhat.com \
--cc=mst@redhat.com \
--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.