From: "Michael S. Tsirkin" <mst@redhat.com>
To: Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org>
Cc: "Philippe Mathieu-Daudé" <philmd@linaro.org>,
"Peter Maydell" <peter.maydell@linaro.org>,
qemu-devel@nongnu.org,
"Gowtham Siddarth" <gowtham.siddarth@arm.com>,
"Marcel Apfelbaum" <marcel.apfelbaum@gmail.com>
Subject: Re: PCIe: SLT attribute mismatch: 0xFF020100 instead of 0x20100
Date: Wed, 30 Aug 2023 06:45:54 -0400 [thread overview]
Message-ID: <20230830064521-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <86c5e07d-58f0-41a0-f497-bd236691188b@linaro.org>
On Wed, Aug 30, 2023 at 10:22:53AM +0200, Marcin Juszkiewicz wrote:
> W dniu 29.08.2023 o 22:46, Michael S. Tsirkin pisze:
> > On Tue, Aug 29, 2023 at 10:44:08PM +0200, Marcin Juszkiewicz wrote:
> > > W dniu 29.08.2023 o 22:40, Michael S. Tsirkin pisze:
> > > > > It passes with sbsa-ref (which is not using QEMU versioning).
> > > > >
> > > > > Fails (as expected) when used new property for each pcie-root-port
> > > > > (ignore line breaks):
> > > > >
> > > > > "-device pcie-root-port,
> > > > > x-pci-express-writeable-slt-bug=true,
> > > > > id=root30,chassis=30,slot=0"
> > > >
> > > > could you also check with -machine pc-q35-8.1 instead of this
> > > > property?
> > >
> > > BSA ACS is AArch64 only.
> >
> > virt-8.1 then
>
> Passes for virt, virt-8.1 and virt-8.0 machines.
>
> ./code/qemu/build/aarch64-softmmu/qemu-system-aarch64 \
> -machine virt-8.0 \
> -m 4096 \
> -cpu neoverse-n1 \
> -smp 2 \
> -drive if=pflash,format=raw,file=QEMU_EFI.fd \
> -drive if=pflash,format=raw,file=QEMU_EFI-vars.fd \
> -drive file=fat:rw:$PWD/disks/virtual/,format=raw \
> -device pcie-root-port,id=root30,chassis=30,slot=0 \
> -device igb,bus=root30 \
> -device qemu-xhci,id=usb \
> -device bochs-display \
> -device e1000e \
> -nographic \
> -usb \
> -device usb-kbd \
> -device usb-tablet \
> -monitor telnet::45454,server,nowait \
> -serial stdio
>
Weird. OK let me post it properly and you can then test and confirm
it fails on 8.1 and 8.0 and passes on latest.
next prev parent reply other threads:[~2023-08-30 10:46 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-29 11:39 PCIe: SLT attribute mismatch: 0xFF020100 instead of 0x20100 Marcin Juszkiewicz
2023-08-29 13:14 ` Peter Maydell
2023-08-29 13:48 ` Michael S. Tsirkin
2023-08-29 14:04 ` Philippe Mathieu-Daudé
2023-08-29 17:07 ` Michael S. Tsirkin
2023-08-29 20:05 ` Marcin Juszkiewicz
2023-08-29 20:18 ` Michael S. Tsirkin
2023-08-29 20:31 ` Marcin Juszkiewicz
2023-08-29 20:40 ` Michael S. Tsirkin
2023-08-29 20:44 ` Marcin Juszkiewicz
2023-08-29 20:46 ` Michael S. Tsirkin
2023-08-30 8:22 ` Marcin Juszkiewicz
2023-08-30 10:45 ` Michael S. Tsirkin [this message]
2023-08-29 13:57 ` Philippe Mathieu-Daudé
2023-08-29 14:43 ` Marcin Juszkiewicz
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=20230830064521-mutt-send-email-mst@kernel.org \
--to=mst@redhat.com \
--cc=gowtham.siddarth@arm.com \
--cc=marcel.apfelbaum@gmail.com \
--cc=marcin.juszkiewicz@linaro.org \
--cc=peter.maydell@linaro.org \
--cc=philmd@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 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).