Re: [PATCH][ATM] add reference counting to atm_dev

Duncan Sands (baldrick@wanadoo.fr)
Fri, 16 May 2003 16:40:43 +0200


On Thursday 15 May 2003 16:32, chas williams wrote:
> In message <20030515052041.GA5995@kroah.com>,Greg KH writes:
> >It's not really bothering me, just wondering when it will go away (I see
> >it when building one of the USB ATM drivers...)
>
> the MOD_* functions in the speedtch driver don't need to be there.
> since 2.3.something (if i remember correctly) the reference counting
> has been handled by the upper layer (ala fops_get/fops_put). the
> following patch removes these extra bits:

Hi Chas, thanks for the patch. I agree that it is correct. However,
I was wondering about ioctl and proc calls. Can the ATM layer call
a driver's ioctl routine before opening a vcc? If so, does it take a
reference to the module first (I didn't spot anything)? Also, are
calls to a driver's proc_read routine protected against module
unloading races (I confess I didn't take the time to look into this,
because my own driver does not sleep in proc_read)?

All the best,

Duncan.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/