lkml.org 
[lkml]   [2026]   [Apr]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] nubus: switch to dynamic root device
On Sat, Apr 25, 2026 at 02:07:10PM +1000, Finn Thain wrote:
> On Fri, 24 Apr 2026, Johan Hovold wrote:
>
> > Driver core expects devices to be dynamically allocated and will, for
> > example, complain loudly if a device that lacks a release function is
> > ever freed.
> >
>
> Yes, in drivers/base/core.c, there is a warning in device_release().
>
> WARN(1, KERN_ERR "Device '%s' does not have a release()
> function, it is broken and must be fixed. See
> Documentation/core-api/kobject.rst.\n",
>
> But there's no way for the refcount for the nubus parent device to reach
> zero that I can see. Did I miss something?

It will if registration ever fails (e.g. due to fault injection or name
collision):

err = device_register(&nubus_parent);
if (err) {
put_device(&nubus_parent);
return err;
}

Johan

\
 
 \ /
  Last update: 2026-04-27 17:40    [from the cache]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and my Meterkast|Read the blog