HEADS UP: destroy_dev_sched() KPI in the tree

Tai-hwa Liang avatar at mmlab.cse.yzu.edu.tw
Sun Jul 8 02:19:35 UTC 2007


On Sat, 7 Jul 2007, Kostik Belousov wrote:
> On Sat, Jul 07, 2007 at 08:36:23PM +0800, Jiawei Ye wrote:
>> On 7/5/07, Kostik Belousov <kostikbel at gmail.com> wrote:
>>>
>>> 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.
>>>
>>>
>> Does this mean that mount_smbfs has to explicitly call it now? It is
>> currently stuck in the devdrn state with the latest kernel :(
>>
>> Jiawei Ye
> Exactly. Patch by Tai-hwa Liang is pending. I also remember that sg(4)

   I'll ask for re@'s approval tomorrow.  Hopefully the new KPI would have
more exposure before my patch is committed.

> had the same problem, but I do not know it current status.

   Though it was reviewed before destroy_dev_sched() KPI enters to the tree,
I'd be appreciate it if you can reviewed the attached patch again.

-- 
Cheers,

Tai-hwa Liang
-------------- next part --------------
--- cam/scsi/scsi_target.c.orig	Wed May  2 11:42:46 2007
+++ cam/scsi/scsi_target.c	Wed May  2 11:43:11 2007
@@ -210,7 +210,7 @@ targclose(struct cdev *dev, int flag, in
 	softc = (struct targ_softc *)dev->si_drv1;
 	if ((softc->periph == NULL) ||
 	    (softc->state & TARG_STATE_LUN_ENABLED) == 0) {
-		destroy_dev(dev);
+		destroy_dev_sched(dev);
 		FREE(softc, M_TARG);
 		return (0);
 	}
@@ -229,7 +229,7 @@ targclose(struct cdev *dev, int flag, in
 			cam_periph_invalidate(softc->periph);
 			softc->periph = NULL;
 		}
-		destroy_dev(dev);
+		destroy_dev_sched(dev);
 		FREE(softc, M_TARG);
 	}
 	cam_periph_unlock(periph);


More information about the freebsd-current mailing list