Date:	Sun, 11 Sep 2011 23:53:19 +0530
From:	Amit Sahrawat <amit.sahrawat83@...il.com>
To:	NamJae Jeon <linkinjeon@...il.com>
Cc:	linux-kernel@...r.kernel.org, linux-fsdevel@...r.kernel.org
Subject: Re: Issue with lazy umount and closing file descriptor in between

There are few things, I am looking at trying out invaliding inode
mappings(address space mapping, flushing out the inode writes - that
will take care of invaliding all the cache entries for that inode and
for any new access it will again go out to disk. Regarding locks, we
need to release the lock in order the 'close the fd', Inode locking
does allows for parallel access - I am looking at its optimal usage if
this can be used.

I will try the cache flushing at first and check its impact.

Thanks & Regards,
Amit Sahrawat

On Sat, Sep 10, 2011 at 4:48 PM, NamJae Jeon <linkinjeon@...il.com> wrote:
> 2011/9/8 Amit Sahrawat <amit.sahrawat83@...il.com>:
>> I know that lazy umount was designed keeping in mind that the
>> mountpoint is not accesible to all future I/O but for the ongoing I/O
>> it will continute to work. It is only after the I/O is finished that
>> the umount will actually occur. But this can be tricky at times
>> considering there are situations where the operation will continue to
>> be executed than what is expected duration, and you cannot unplug the
>> device during that period because there are chances of filesystem
>> corruption on doing so.
>> Is there anything which could be done in this context? because simply
>> reading the fd-table and closing fd's will not serve the purpose and
>> there is every chance of a OOPs occuring due to this closing.
>> Signalling from this point to the all the process's with open fd's on
>> that mountpoint to close fd i.e., handling needs to be done from the
>> user space applications...? Does this make sense
>>
>> Please through some insight into this. I am not looking for exact
>> solution it is just mere opinion's on this that can add to this.
>>
>> Thanks & Regards,
>> Amit Sahrawat
>>
>> On Tue, Sep 6, 2011 at 10:56 PM, Amit Sahrawat
>> <amit.sahrawat83@...il.com> wrote:
>>> We have observed below issues in busybox umount:
>>> 1. force umount (umount -f): it does not work as expected.
>>> 2. lazy umount(umount -l): it detaches the mount point but waits for
>>> current mount point users(processes) to finish.
>>> Corruption happens when we powerdown, while lazy umount is waiting for
>>> a process to finish.
>>> (e.g. #dd if=/dev/zero of=/mnt/test.txt ).
>>> What could be the ideal way so as to avoid file system corruption in
>>> above scenario?
>>> Is it fine to close all open file descriptors on umount system call
>>> before attempting umount? But this results in OOPS in certain
>>> situations like:
>>> 1. User app issue a write/read request
>>> 2. Write reaches in kernel space but sleeps for some time e.g. it is
>>> not available in dentry cache.
>>> 3. In the meanwhile, we issue umount. This will close open file
>>> descriptor, free file/dentry object and then umount.
>>> 4. Now write wakes up and finds NULL file/dentry object and triggers oops.
>>> Please offer some advice on this issue.
>>> Thanks & Regards,
>>> Amit Sahrawat
>>>
>>
>
> Before close opend file, plz try to flush write request using
> sys_fsync or etc.. and you should do mutex_lock about opened inode at
> the same time. because next write request should be blocked by user
> app.
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists