Messages in this thread | | | Date | Tue, 26 Aug 2003 16:09:54 +0700 | From | Samphan Raruenrom <> | Subject | Re: [PATCH] Add MOUNT_STATUS ioctl to cdrom device |
| |
Christoph Hellwig wrote: >>(my random idea) >>- fcntl(open("/dev/cdrom", F_MNTSTAT) >>- umount2("/dev/cdrom", MS_TEST) // not actually perform >>- new system call! e.g. mntstat(open("/dev/cdrom")) > In userspace. Or you could tell me what you want to actually > archive - your call by itself doesn't make any sense.
To get the same result in userspace, it means scanning /proc like fuser, right? Do you think it is ok to do that? Because the daemon (patched magicdev) poll the cdrom device very often (every 2 sec.) That's why I thought that faster is better (less load to the system) and push the job to kernel-space.
This is what I really want to achive - "enable the eject button, by locking the drive only when the device is really in use, unlock otherwise".
I implement this as a modified version of GNOME magicdev. http://bugzilla.gnome.org/show_bug.cgi?id=119892
The only visible feature of this new magicdev is that now GNOME users can eject there CDs (the discs' icon will disappear). The eject button now act as 'umount' command.
One new requirement from this new magicdev is the question "will umount failed?". I have no preference on any way to implement it. Should there be the right way to do it, I'll do so. I can think of many way to implement it (including adding a new lazy-lock mode to cdrom device) but since I have no kernel hacking experience, I need everyone advices. Novice users need this 'eject' button after all.
-- Samphan Raruenrom, The Open Source Project, National Electronics and Computer Technology Center, National Science and Technology Development Agency, Thailand.
- 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/
| |