From: "Pasi Kärkkäinen" <pasik@iki.fi>
To: Jan Beulich <jbeulich@suse.com>
Cc: xen-devel <xen-devel@lists.xenproject.org>,
"Marek Marczykowski-Górecki" <marmarek@invisiblethingslab.com>
Subject: Re: [Xen-devel] Broken PCI device passthrough, after XSA-302 fix?
Date: Mon, 20 Jan 2020 11:10:30 +0200 [thread overview]
Message-ID: <20200120091029.GP21578@reaktio.net> (raw)
In-Reply-To: <f4099f0e-2084-1fb3-5038-9e82b6a3e3e1@suse.com>
On Mon, Jan 20, 2020 at 09:36:27AM +0100, Jan Beulich wrote:
> On 19.01.2020 11:39, Pasi Kärkkäinen wrote:
> > On Mon, Jan 06, 2020 at 02:06:14PM +0100, Jan Beulich wrote:
> >> On 04.01.2020 02:07, Marek Marczykowski-Górecki wrote:
> >>> I have a multi-function PCI device, behind a PCI bridge, that normally
> >>> I assign to a single domain. But now it fails with:
> >>>
> >>> (XEN) [VT-D]d14: 0000:04:00.0 owned by d0!<G><0>assign 0000:05:00.0 to dom14 failed (-22)
> >>
> >> I've tried this out in as close a setup as I could arrange for, but
> >> not one matching your scenario. I didn't find a system with a
> >> suitably placed (in the topology) multi-function device (had to use
> >> a single-function one), and of course I did this on (close to)
> >> master. No anomalies. Hence I wonder whether either of the two
> >> differences mentioned matters, and - if, as I suspect, it's the
> >> multi-function aspect that is relevant here - how things would have
> >> worked at all before those recent changes. This is because I think
> >> you should have hit the same error path even before, and it would
> >> seem to me that the patch below might be (and have been) needed.
> >>
> >
> > I think Marek confirmed in the other mail that this patch fixes the issue.
> >
> > Are you planning to merge this patch?
>
> Well, it is still pending a maintainer ack. Kevin has requested a
> (mechanical) change just over the weekend.
>
Ah, sorry, I missed the other thread. The patch in question seems to be:
"[PATCH 1/2] VT-d: don't pass bridge devices to domain_context_mapping_one()",
in thread: "[PATCH 0/2] VT-d: domain_context_mapping_one() adjustments".
Thanks.
-- Pasi
>
> Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
prev parent reply other threads:[~2020-01-20 9:11 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-04 1:07 [Xen-devel] Broken PCI device passthrough, after XSA-302 fix? Marek Marczykowski-Górecki
2020-01-06 11:18 ` Jan Beulich
2020-01-06 14:04 ` Marek Marczykowski-Górecki
2020-01-06 14:16 ` Marek Marczykowski-Górecki
2020-01-06 14:29 ` Marek Marczykowski-Górecki
2020-01-06 14:38 ` Jan Beulich
2020-01-06 16:37 ` Marek Marczykowski-Górecki
2020-01-06 13:06 ` Jan Beulich
2020-01-19 10:39 ` Pasi Kärkkäinen
2020-01-20 8:36 ` Jan Beulich
2020-01-20 9:10 ` Pasi Kärkkäinen [this message]
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=20200120091029.GP21578@reaktio.net \
--to=pasik@iki.fi \
--cc=jbeulich@suse.com \
--cc=marmarek@invisiblethingslab.com \
--cc=xen-devel@lists.xenproject.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.