Re: [PATCH] Wake up mandatory locks waiter on chmod (v2)

On Mon, 2007-09-17 at 18:16 +0400, Pavel Emelyanov wrote:
> Trond Myklebust wrote:
> > On Mon, 2007-09-17 at 12:13 +0400, Pavel Emelyanov wrote:
> >> When the process is blocked on mandatory lock and someone changes 
> >> the inode's permissions, so that the lock is no longer mandatory,
> >> nobody wakes up the blocked process, but probably should.
> > 
> > Please explain in more detail why we need this patch.
> 
> From "this fixes an OOPs/deadlock/leak" POV we do not. This is
> just an attempt to make the locking code be more consistent and
> clean.

Why do you think we get a deadlock or leak? AFAICS if the user turns off
mandatory locks on the file, then the existing locks default back into
advisory locks which use the same notification mechanism as the
mandatory locks.

IOW: the process that is waiting in locks_mandatory_area() will be
released as soon as the advisory lock is dropped. If that theory is
broken in practice, then that is the bug that we need to fix. We neither
want to add a load of locking crap to notify_change(), nor should we
need to.

Cheers
   Trond

-
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/

---

• Follow-Ups:
  • Re: [PATCH] Wake up mandatory locks waiter on chmod (v2)
    • From: Pavel Emelyanov <[email protected]>

• References:
  • [PATCH] Wake up mandatory locks waiter on chmod (v2)
    • From: Pavel Emelyanov <[email protected]>
  • Re: [PATCH] Wake up mandatory locks waiter on chmod (v2)
    • From: Trond Myklebust <[email protected]>
  • Re: [PATCH] Wake up mandatory locks waiter on chmod (v2)
    • From: Pavel Emelyanov <[email protected]>

• Prev by Date: git-send-email creates duplicate Message-Id's
• Next by Date: [PATCH] ext34: ensure do_split leaves enough free space in both blocks
• Previous by thread: Re: [PATCH] Wake up mandatory locks waiter on chmod (v2)
• Next by thread: Re: [PATCH] Wake up mandatory locks waiter on chmod (v2)
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]