All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Jamin Lin <jamin_lin@aspeedtech.com>
Cc: Junjie Cao <junjie.cao@linux.dev>,
	"kraxel@redhat.com" <kraxel@redhat.com>,
	"marcandre.lureau@redhat.com" <marcandre.lureau@redhat.com>,
	"clg@kaod.org" <clg@kaod.org>,
	"pierrick.bouvier@oss.qualcomm.com"
	<pierrick.bouvier@oss.qualcomm.com>,
	"peter.maydell@linaro.org" <peter.maydell@linaro.org>,
	Kane Chen <kane_chen@aspeedtech.com>,
	"philmd@oss.qualcomm.com" <philmd@oss.qualcomm.com>,
	"kfting@nuvoton.com" <kfting@nuvoton.com>,
	"wuhaotsh@google.com" <wuhaotsh@google.com>,
	"zhao1.liu@intel.com" <zhao1.liu@intel.com>,
	"richard.henderson@linaro.org" <richard.henderson@linaro.org>,
	"deller@gmx.de" <deller@gmx.de>,
	"sstabellini@kernel.org" <sstabellini@kernel.org>,
	"anthony@xenproject.org" <anthony@xenproject.org>,
	"edgar.iglesias@gmail.com" <edgar.iglesias@gmail.com>,
	Troy Lee <troy_lee@aspeedtech.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	"qemu-arm@nongnu.org" <qemu-arm@nongnu.org>,
	"xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>
Subject: Re: [PATCH v8 02/18] hw/usb/hcd-xhci: Add an auto-hub property and leave it off
Date: Fri, 6 Nov 2026 09:08:28 +0000	[thread overview]
Message-ID: <au2ZDOY7MGdY74L9@redhat.com> (raw)
In-Reply-To: <SEZPR06MB7362D089282550D7585C16DEFC762@SEZPR06MB7362.apcprd06.prod.outlook.com>

On Fri, Nov 06, 2026 at 03:45:58AM +0000, Jamin Lin wrote:
> > Subject: RE: [PATCH v8 02/18] hw/usb/hcd-xhci: Add an auto-hub property and
> > leave it off
> > 
> > > Subject: Re: [PATCH v8 02/18] hw/usb/hcd-xhci: Add an auto-hub
> > > property and leave it off
> > >
> > > Hi Jamin,
> > >
> > > On Mon, Nov 02, 2026 at 09:22:05AM +0000, Jamin Lin wrote:
> > > > Add auto-hub and leave it off. hw_compat_11_1 turns it on, so an
> > > > older machine type keeps the hub it has today and a guest saved by
> > > > an older QEMU still loads.
> > >
> > > Reran the migration check from v7: with pc-q35-11.1, a guest saved
> > > without the series loads on v8 with the hub in place, for qemu-xhci
> > > with four HID devices and for q35 -usb with six.
> > >
> > > But this missed the 11.2 soft freeze, so in 12.0 pc-q35-11.2 is an old
> > > machine type too, and hw_compat_11_1 (my suggestion on v7) doesn't
> > > cover it. Saved with pc-q35-11.2 on master, both cases fail on v8:
> > >
> > >   Unknown section or instance '0000:00:01.0/4/usb-hub' 0
> > >   Unknown section or instance '0000:00:1d.7/6/usb-hub' 0
> > >
> > > The entries from 02-05 want hw_compat_11_2 once the 12.0 machine types
> > > are in.
> > >
> > > Junjie
> > 
> > Hi Junjie, Marc-André, and Gerd,
> > 
> > Due to the QEMU code freeze and the planned 11.2 release in December,
> 
> 
> I am not sure about the correct release version. I expect it to be v11.2.
> If the version is wrong, please correct me.

Yes, the forthcoming release is 11.2, the version then ticks over to 12.0
for the first release of 2027.

> > disabling automatic hub creation while keeping migration compatibility with
> > older QEMU versions requires adding the property to hw_compat_11_2.
> > We need to wait until after the QEMU 11.2 release for the maintainers to add
> > hw_compat_11_2 to each machine type.
> > 
> > The USB redirection server is my original upstream goal, and I think this task is
> > independent of the automatic hub task.
> > So, I want to split the auto-hub changes into a separate prerequisite patch
> > series and send v9 after the QEMU 11.2 release.
> > 
> > The USB redirection server common protocol series still needs rework on the
> > device announcement and control transfer patches.
> > I will send v9 next week and mention that it requires the patch series that
> > disables automatic hub creation.
> > 
> > This way, we can continue testing and reviewing the USB redirection server
> > common protocol this month.(November)
> > 
> > "If you agree, I will send v9 next week. Otherwise, I will wait until after the
> > QEMU 11.2 release and then resend the full patch series as v9."

Don't delay sending new versions of your series because of the release
freeze.

Just put a "TODO: change to hw_compat_11_2 before merge" in the commit
message to call out the bit that you know needs changing before this
can be finally ready to merge.

With regards,
Daniel
-- 
|: https://berrange.com       ~~        https://hachyderm.io/@berrange :|
|: https://libvirt.org          ~~          https://entangle-photo.org :|
|: https://pixelfed.art/berrange   ~~    https://fstop138.berrange.com :|



  reply	other threads:[~2026-11-06  9:09 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-11-02  9:22 [PATCH v8 00/18] hw/usb: Add a usbredir server transport Jamin Lin
2026-11-02  9:22 ` [PATCH v8 01/18] hw/usb/bus: Add an auto-hub flag to the USB bus Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-02  9:22 ` [PATCH v8 02/18] hw/usb/hcd-xhci: Add an auto-hub property and leave it off Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-03 14:23   ` Junjie Cao
2026-11-06  3:29     ` Jamin Lin
2026-11-06  3:45       ` Jamin Lin
2026-11-06  9:08         ` Daniel P. Berrangé [this message]
2026-11-06 11:30           ` Junjie Cao
2026-11-02  9:22 ` [PATCH v8 03/18] hw/usb/hcd-ehci: " Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-02  9:22 ` [PATCH v8 04/18] hw/usb/hcd-uhci: " Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-02  9:22 ` [PATCH v8 05/18] hw/usb/hcd-ohci: " Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-03  1:29     ` Jamin Lin
2026-11-02  9:22 ` [PATCH v8 06/18] hw/usb/hcd-dwc2: " Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-03 14:23   ` Junjie Cao
2026-11-02  9:22 ` [PATCH v8 07/18] hw/usb/xen-usb: Do not insert an automatic hub Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-02  9:22 ` [PATCH v8 08/18] hw/usb/bus: Turn off the automatic hub by default Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-02  9:22 ` [PATCH v8 09/18] hw/arm/npcm7xx: Keep the automatic hub Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-03  1:06     ` Jamin Lin
2026-11-03 14:23   ` Junjie Cao
2026-11-02  9:22 ` [PATCH v8 10/18] docs/system/devices/usb: Note the automatic hub change Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-03 14:23   ` Junjie Cao
2026-11-06  3:31     ` Jamin Lin
2026-11-02  9:22 ` [PATCH v8 11/18] hw/usb/redirect-server: Add a usbredir server device Jamin Lin
2026-11-03 14:23   ` Junjie Cao
2026-11-02  9:22 ` [PATCH v8 12/18] hw/usb/redirect-server: Connect usbredirparser to a chardev Jamin Lin
2026-11-03 14:23   ` Junjie Cao
2026-11-02  9:22 ` [PATCH v8 13/18] hw/usb/redirect-server: Announce the exported device Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-03  7:14     ` Jamin Lin
2026-11-02  9:22 ` [PATCH v8 14/18] hw/usb/redirect-server: Implement control transfers Jamin Lin
2026-11-03 14:23   ` Junjie Cao
2026-11-02  9:22 ` [PATCH v8 15/18] hw/usb/redirect-server: Implement bulk and interrupt transfers Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-03 14:23   ` Junjie Cao
2026-11-02  9:22 ` [PATCH v8 16/18] hw/usb/redirect-server: Stream interrupt IN endpoints Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-02  9:22 ` [PATCH v8 17/18] hw/usb/redirect-server: Implement isochronous OUT transfers Jamin Lin
2026-11-02 14:07   ` marcandre.lureau
2026-11-03 14:23   ` Junjie Cao
2026-11-02  9:22 ` [PATCH v8 18/18] hw/arm/aspeed: Enable the usbredir server transport Jamin Lin

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=au2ZDOY7MGdY74L9@redhat.com \
    --to=berrange@redhat.com \
    --cc=anthony@xenproject.org \
    --cc=clg@kaod.org \
    --cc=deller@gmx.de \
    --cc=edgar.iglesias@gmail.com \
    --cc=jamin_lin@aspeedtech.com \
    --cc=junjie.cao@linux.dev \
    --cc=kane_chen@aspeedtech.com \
    --cc=kfting@nuvoton.com \
    --cc=kraxel@redhat.com \
    --cc=marcandre.lureau@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@oss.qualcomm.com \
    --cc=pierrick.bouvier@oss.qualcomm.com \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    --cc=sstabellini@kernel.org \
    --cc=troy_lee@aspeedtech.com \
    --cc=wuhaotsh@google.com \
    --cc=xen-devel@lists.xenproject.org \
    --cc=zhao1.liu@intel.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 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.