Re: [PATCH v2 7/8] rust: pci: add typed SR-IOV PF registration data
From: Danilo Krummrich
Date: Tue Sep 29 2026 - 04:39:38 EST
On Tue Sep 29, 2026 at 9:27 AM CEST, Zhi Wang wrote:
> On Mon, 28 Sep 2026 22:08:46 +0200
> "Danilo Krummrich" <dakr@xxxxxxxxxx> wrote:
>> The published thing seems unnecessary, am I missing something?
>
> I was thinking of PF drivers that only need to enable SR-IOV and do
> not need to share any data or services with their VF drivers.
>
> If we rely on VfRegistration to disable SR-IOV on PF removal, would
> those drivers also need to keep a VfRegistration with () as its data,
> solely for the teardown guarantee?
For this case we could either build a new guard type on top, or just add a
separate one to avoid returning an impl PinInit. But I'd wait until we have a
use-case for this.
As for the existing 'published' logic in this patch, I'm still confused. It
can't ever be false to begin with, so it seems it doesn't do anything?