lkml.org 
[lkml]   [2000]   [Sep]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] Useless inode semaphore locking in 2.4.0-test8
    On Fri, Sep 15, 2000 at 02:00:38PM +0200, Eric PAIRE wrote:
    > Hi,
    >
    > In open.c:do_truncate(), the call to notify_change() is protected by
    > the inode->i_sem, which seems to me useless, and thus can be removed.
    > BTW, I also removed the useless inode pointer and error local variables.

    Please don't. You opened race country with writes in most file systemsnow.
    In reiserfs it is also needed to lock against tail merging.

    Also truncate inode locking is needed to get a halfway reliable loopback
    device (unlike the current one)

    -Andi

    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2005-03-22 12:38    [W:3.545 / U:0.248 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site