Page MenuHomeFreeBSD

Don't do the busy dance in icee_open/close

Authored by imp on Sep 14 2020, 10:36 PM.
Referenced Files
Unknown Object (File)
Dec 23 2023, 1:15 AM
Unknown Object (File)
Dec 16 2023, 9:52 AM
Unknown Object (File)
Sep 2 2023, 4:47 AM
Unknown Object (File)
Jul 12 2023, 3:06 PM
Unknown Object (File)
Jul 11 2023, 9:50 PM
Unknown Object (File)
Jul 8 2023, 11:20 AM
Unknown Object (File)
Apr 9 2023, 1:04 PM
Unknown Object (File)
Mar 29 2023, 3:01 PM



We don't need to do the busy dance for this driver. It's handled by
destroy_dev() entirely. Since all we did was busy/unbusy in
open/close, just delete them. We therefore don't need to track closes

Diff Detail

rS FreeBSD src repository - subversion
Lint Not Applicable
Tests Not Applicable

Event Timeline

imp requested review of this revision.Sep 14 2020, 10:36 PM
imp created this revision.
ian added a subscriber: ian.

Applied patch to freebsd 8 (which we use at $work but it has the icee.c source from -current) and freebsd 11 and it behaves as expected on our products.

Based on the commit history, I was the one who added the busy/unbusy, and I think I was just being a bit too overprotective, wanting to prevent doing a kldunload on the module while something had the cdev open. The TRACKCLOSE was just my misunderstanding of how that flag works, it was never needed.

This revision is now accepted and ready to land.Sep 14 2020, 11:16 PM
This revision was automatically updated to reflect the committed changes.