lkml.org 
[lkml]   [2008]   [Nov]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH RESEND] char_dev: add cdev->release() and convert cdev_alloc() to use it
Tejun Heo wrote:
> (lkml was missing, adding it back and quoting whole body for reference)
>
> Tejun Heo wrote:
>> Greg KH wrote:
>>> On Mon, Nov 17, 2008 at 06:18:45PM +0900, Tejun Heo wrote:
>>>> (this is a private message)
>>>>
>>>> PING. :-)
>>> Heh.
>>>
>>> I've lost this out of my tree, as I didn't know you still wanted this
>>> applied. If you need it for your char-fuse code, then I don't have an
>>> objection to it, and you can take it through your tree.
>>>
>>> Or do you want me to take it through mine?
>>>
>>> Your choice :)
>> I'll push it through Miklos' tree w/ your ACK then.
>>
>> Thanks. :-)
>>
>
>

Tejun Hi.

I just saw this thread for the first time and it left me confused.
What was the final verdict. Is this patch going in at the end?
Which incarnation of it? is there a public git tree I can try?

The reason I ask is because I have just the same principal work in one of
my test trees. What I have is a Filesystem, osdfs, that is mounted
on an OSD scsi-device, which is a char-device. Now the osdfs when mounting
an OSD device does not use __open, like user mode it needs some kernel
reference counting to keep the char-device up. On the other hand
the actual teardown and unmap of the char-device is done from the scsi-ml
remove vector. So just like in sd, sr and other scsi ULDs I need to unmap
the device but keep the memory allocated and available until the last reference.
All this is usually done using the Release() of the block-device. But for me
I only have a char-device. Currently what I had to do is keep another kref
to govern the device's lifecycle and sync every thing together. A Release() at
the char-dev would let me reuse what's there and let me clean all that code up.

While Investigating the problem and compering what was done on the block-device
side, I've seen more then a few places that private reference counting could be
dropped completely, and the char-dev could be used. Off my head some of these places
are:
- UBI used by UBIFS
- sg.c which does not have a Kernel user but needs it's char device until
scsi-Remove and/or __close()

and other places as well.

Thanks for doing this
Boaz


\
 
 \ /
  Last update: 2008-11-18 14:47    [W:0.122 / U:0.036 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site