Re: [PATCH 1/1] rust: i2c: fix adapter refcounting during unregistration

["Nicolás Antinori" <nico.antinori.7@gmail.com>]

Hello!
On Wed Nov 11, 2026 at 12:06 PM -03, Igor Korotin wrote:
> Hi Priya, Nicolás,
>
> Priya, nice catch, and a solid PoC.
>
> Nicolás -- this steps on your inc_ref patch from September (still
> waiting on Andi). Same function, same root problem really.
>
> I prefer Priya's version. No new C export, no Ack needed from anyone,
> just __module_get()/get_device() directly on the pointer we already
> know is valid -- and it kills your locking concern too as a side
> effect.
>
> Didn't want this to land without you seeing it first. Shout if you
> disagree.

Yes, I think this version is better too. In fact the solution is the
same as the v1 I sent a while back [1] (but Sashiko caught the
CONFIG_MODULE_UNLOAD config problem). My idea was to send that version
fixed (would ended up the same as Priya's patch) but there was a
discussion about adding the C API [2].

Go with this patch, no problem on that!

Best Regards,
Nicolás

[1] https://lore.kernel.org/all/20260615201141.8920-1-nico.antinori.7@gmail.com/T/#u
[2] https://lore.kernel.org/all/DJBDWVUWH3OT.1GDEIJT0R4UTK@garyguo.net/