> On May 16, 2016, at 09:47, Hans Petter Selasky <h...@selasky.org> wrote:
>
> On 05/16/16 18:31, Ngie Cooper (yaneurabeya) wrote:
>> Dumb question — couldn’t we run the check without locking Giant, then delete
>> the child, e.g.
>>
>> if (bsddev != NULL) {
>> mtx_lock(&Giant);
>> device_delete_child(device_get_parent(bsddev), bsddev);
>> mtx_unlock(&Giant);
>> }
>> put_device(dev);
>
> I guess so. Does it make a difference for you?
First off, how often does the bsddev == NULL case occur?
If it doesn’t occur often, doing this increases contention on Giant
unnecessarily…
Thanks,
-Ngie
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
