HEADS UP: destroy_dev_sched() KPI in the tree
Kostik Belousov
kostikbel at gmail.com
Thu Jul 5 14:45:13 UTC 2007
On Thu, Jul 05, 2007 at 05:18:43PM +0300, Danny Braniss wrote:
>
> > Today, I got re@ permission and committed the destroy_dev_sched() KPI
> > into the tree. As suggested by Nate Lawson, destroy_dev() tries to
> > auto-detect "devdrn" deadlock condition and transforms destroy_dev()
> > from d_close() csw method into destroy_dev_sched(). I together with
> > Peter Holm tried to test the change as thoroughly as possible.
> >
> > The obvious problematic area are the races between device driver
> > module unload and destroy_dev_sched(). At least smb(4) definitely
> > has that race that shall be worked around by draining events with
> > drain_dev_clone_events() and destroy_dev_drain(&cdevsw); see snp(4)
> > change for example.
> >
> > Please, report issues caused by the patch to the list with me Cc:ed.
> > Change of autodetection of deadlock in destroy_dev() was made in separate
> > commit for ease of reverting.
>
> and today I got a chance to check it, and it works for me (iscsi_initiator).
Today, I reverted the part of commit that tried to transform destroy_dev()
from d_close() into destroy_dev_sched(). See kern_conf.c, rev. 1.208.
You shall call destroy_dev_sched() explicitely.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-current/attachments/20070705/b2d7dc96/attachment.pgp
More information about the freebsd-current
mailing list