On Saturday 03 September 2005 23:06, Joel Becker wrote:
> dlmfs is *tiny*. The VFS interface is less than his claimed 500
> lines of savings.
It is 640 lines.
> The few VFS callbacks do nothing but call DLM
> functions. You'd have to replace this VFS glue with sysfs glue, and
> probably save very few lines of code.
> In addition, sysfs cannot support the dlmfs model. In dlmfs,
> mkdir(2) creates a directory representing a DLM domain and mknod(2)
> creates the user representation of a lock. sysfs doesn't support
> mkdir(2) or mknod(2) at all.
I said "configfs" in the email to which you are replying.
> More than mkdir() and mknod(), however, dlmfs uses open(2) to
> acquire locks from userspace. O_RDONLY acquires a shared read lock (PR
> in VMS parlance). O_RDWR gets an exclusive lock (X). O_NONBLOCK is a
> trylock. Here, dlmfs is using the VFS for complete lifetiming. A lock
> is released via close(2). If a process dies, close(2) happens. In
> other words, ->release() handles all the cleanup for normal and abnormal
> termination.
>
> sysfs does not allow hooking into ->open() or ->release(). So
> this model, and the inherent lifetiming that comes with it, cannot be
> used.
Configfs has a per-item release method. Configfs has a group open method.
What is it that configfs can't do, or can't be made to do trivially?
> If dlmfs was changed to use a less intuitive model that fits
> sysfs, all the handling of lifetimes and cleanup would have to be added.
The model you came up with for dlmfs is beyond cute, it's downright clever.
Why mar that achievement by then failing to capitalize on the framework you
already have in configfs?
By the way, do you agree that dlmfs is too inefficient to be an effective way
of exporting your dlm api to user space, except for slow-path applications
like you have here?
Regards,
Daniel
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
|
|