From: Mike Lothian <mike@fireburn.co.uk>
To: linux-usb@vger.kernel.org
Cc: "Mike Lothian" <mike@fireburn.co.uk>,
"Miguel Ojeda" <ojeda@kernel.org>,
"Boqun Feng" <boqun@kernel.org>, "Gary Guo" <gary@garyguo.net>,
"Björn Roy Baron" <bjorn3_gh@protonmail.com>,
"Benno Lossin" <lossin@kernel.org>,
"Andreas Hindborg" <a.hindborg@kernel.org>,
"Alice Ryhl" <aliceryhl@google.com>,
"Trevor Gross" <tmgross@umich.edu>,
"Danilo Krummrich" <dakr@kernel.org>,
"Daniel Almeida" <daniel.almeida@collabora.com>,
"Tamir Duberstein" <tamird@kernel.org>,
"Alexandre Courbot" <acourbot@nvidia.com>,
"Onur Özkan" <work@onurozkan.dev>,
"Nathan Chancellor" <nathan@kernel.org>,
"Nick Desaulniers" <nick.desaulniers+lkml@gmail.com>,
"Bill Wendling" <morbo@google.com>,
"Justin Stitt" <justinstitt@google.com>,
rust-for-linux@vger.kernel.org, llvm@lists.linux.dev
Subject: [PATCH v3 0/5] rust: usb: host-side abstractions for a bulk-endpoint driver
Date: Wed, 26 Aug 2026 17:30:36 +0100 [thread overview]
Message-ID: <20260826163101.4168-1-mike@fireburn.co.uk> (raw)
Host-side USB abstractions for a driver whose device is a bulk-endpoint pipe
rather than a class device
Revocable typed interface I/O, so an interface cannot be used after the core
has taken it back
Reusable URBs and persistent bulk queues, which is what keeps a video stream
in flight without allocating per transfer
The device descriptor fields a driver needs to identify hardware before it
decides to drive it, and a queue-readiness check
A device id constructor matching on vendor together with interface class,
subclass and protocol, for a composite device whose function is not
identified by the product id alone
Letting a driver keep its interface usable while unbinding, so teardown can
still talk to the device it is releasing
Changes since v2:
v2 10/11, "keep usb::Device private and gate ...", is dropped entirely.
Oliver Neukum was right that it was conceptually wrong: USB does device
level operations, and hiding that behind an interface is a layering
violation. Device stays public
as_bound() is gone from both the binding and the driver. Danilo Krummrich's
point stands: needing an unsafe as_bound() means the design or the
infrastructure is wrong, not that the escape hatch is needed
reset_configuration() is gone, and set_interface() now exists in two
correctly scoped forms, one on Interface<Bound> taking an altsetting and
one on Device taking interface plus altsetting
What replaces the concealment is lifecycle gating: an adapter-owned,
revocable I/O window that is valid across probe, suspend, reset, resume and
disconnect and invalid outside them. That is the interval in which I/O is
legal, which is narrower than "the interface is bound"
There is no private URB implementation. Colin Braun's URB RFC is carried
unchanged as the foundation and this builds on it
A topology walk and a device-removal notifier were written after v2 and are
not here. They existed for a second consumer that is not part of this
posting, so nothing in what is sent would call them
Alan Stern's lifecycle point is what makes device access from an interface
sound, and is worth restating because the whole shape depends on it: an
unconfigured device has no interfaces, so an interface that exists implies a
configured device
v2: https://lore.kernel.org/r/20260703030020.2694-1-mike@fireburn.co.uk
The rest of the posting, which is one series per subsystem:
rust-core, 9 patches, rust-for-linux and linux-kernel
https://lore.kernel.org/r/20260826162851.2497-1-mike@fireburn.co.uk
rust-crypto, 2 patches, linux-crypto and rust-for-linux
https://lore.kernel.org/r/20260826163004.3365-1-mike@fireburn.co.uk
rust-usb, 5 patches, this one
rust-drm, 23 patches, to dri-devel and rust-for-linux, not sent yet
rust-firmware, 1 patch, to linux-kernel and rust-for-linux, not sent yet
drm-vino, 13 patches, to dri-devel, not sent yet
Vino is the user for all of them. The abstractions themselves are generic and
carry no knowledge of DisplayLink
The whole thing is one branch, base and prerequisites included, which is the
quickest way to read it:
git clone -b vino-v3 https://github.com/FireBurn/linux
cd linux
make LLVM=1 rustavailable
make LLVM=1 -j$(nproc)
make LLVM=1 -j$(nproc) modules
CONFIG_RUST=y and CONFIG_DRM_VINO=m are the two to set; DRM_VINO selects the
rest of what it needs
It is the exact tree these patches were generated from, at 4c9ba407018e, the
drm-rust-next tip of 2026-08-06. drm-next has moved on since, and this follows
drm-rust-next deliberately: the KMS layer underneath this work lives only there,
and that tree picks up drm-next on its own schedule
Two commits on the branch are not in any of the series above, because they
enable no part of Vino: a scheduler call site that stops compiling under the
locking-guard series, and the Kms associated type Tyr needs once the KMS
registration trait requires one
It applies to the base above plus this, and nothing else:
Colin Braun, rust: usb: add usb request block abstractions
https://lore.kernel.org/r/20260712-urb-abstraction-v1-v1-0-9fa011634ead@gmail.com
The reference branch also carries Boqun Feng's counted interrupt disabling
series, which SpinLockIrq needs. One patch of it is already in tip locking/core
as e901c1510e24
These patches were written with the assistance of Claude (Anthropic), used
through Claude Code as an interactive coding assistant, across the design, the
implementation and the tests. Every patch it contributed to carries an
Assisted-by trailer. The Signed-off-by is mine: I have reviewed and tested what
is here and I stand behind it
Mike Lothian (5):
rust: usb: add revocable typed interface I/O
rust: usb: add reusable URBs and persistent bulk queues
rust: usb: expose device descriptor fields and queue readiness
rust: usb: add a vendor-and-interface-info device id constructor
rust: usb: let a driver keep its interface usable while unbinding
3 files changed, 1553 insertions(+), 26 deletions(-)
base-commit: 4c9ba407018e8deb06dbc643112bac8f40404f95
prerequisite-message-id: <20260712-urb-abstraction-v1-v1-0-9fa011634ead@gmail.com>
next reply other threads:[~2026-08-26 16:31 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-26 16:30 Mike Lothian [this message]
2026-08-26 16:30 ` [PATCH v3 1/5] rust: usb: add revocable typed interface I/O Mike Lothian
2026-08-26 18:59 ` Danilo Krummrich
2026-08-26 16:30 ` [PATCH v3 2/5] rust: usb: add reusable URBs and persistent bulk queues Mike Lothian
2026-08-26 16:30 ` [PATCH v3 3/5] rust: usb: expose device descriptor fields and queue readiness Mike Lothian
2026-08-26 16:30 ` [PATCH v3 4/5] rust: usb: add a vendor-and-interface-info device id constructor Mike Lothian
2026-08-26 16:30 ` [PATCH v3 5/5] rust: usb: let a driver keep its interface usable while unbinding Mike Lothian
2026-08-26 18:25 ` [PATCH v3 0/5] rust: usb: host-side abstractions for a bulk-endpoint driver Danilo Krummrich
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=20260826163101.4168-1-mike@fireburn.co.uk \
--to=mike@fireburn.co.uk \
--cc=a.hindborg@kernel.org \
--cc=acourbot@nvidia.com \
--cc=aliceryhl@google.com \
--cc=bjorn3_gh@protonmail.com \
--cc=boqun@kernel.org \
--cc=dakr@kernel.org \
--cc=daniel.almeida@collabora.com \
--cc=gary@garyguo.net \
--cc=justinstitt@google.com \
--cc=linux-usb@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=lossin@kernel.org \
--cc=morbo@google.com \
--cc=nathan@kernel.org \
--cc=nick.desaulniers+lkml@gmail.com \
--cc=ojeda@kernel.org \
--cc=rust-for-linux@vger.kernel.org \
--cc=tamird@kernel.org \
--cc=tmgross@umich.edu \
--cc=work@onurozkan.dev \
/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