linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Scott Lovenberg <scott.lovenberg@gmail.com>
To: Jeff Layton <jlayton@redhat.com>
Cc: Jeremy Allison <jra@samba.org>,
	Andreas Dilger <adilger@dilger.ca>,
	"linux-fsdevel@vger.kernel.org Devel"
	<linux-fsdevel@vger.kernel.org>,
	Ganesha NFS List <nfs-ganesha-devel@lists.sourceforge.net>,
	"samba-technical@lists.samba.org"
	<samba-technical@lists.samba.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [RFC PATCH 0/5] locks: implement "filp-private" (aka UNPOSIX) locks
Date: Sat, 12 Oct 2013 16:56:21 -0400	[thread overview]
Message-ID: <CAFB9KM2avLpWaMhzZVnB0oXXzWb0NSOBB6oR+RE9JPeuustmvA@mail.gmail.com> (raw)
In-Reply-To: <20131011204244.72c2d672@corrin.poochiereds.net>

On Fri, Oct 11, 2013 at 8:42 PM, Jeff Layton <jlayton@redhat.com> wrote:
> On Fri, 11 Oct 2013 20:18:58 -0400
> Scott Lovenberg <scott.lovenberg@gmail.com> wrote:
>
>>
>> On Oct 11, 2013, at 19:49, Jeremy Allison <jra@samba.org> wrote:
>>
>> > On Fri, 11 Oct 2013 15:36:43 -0600 Andreas Dilger <adilger@dilger.ca> wrote:
>> >>>
>> >>> At this point, my main questions are:
>> >>>
>> >>> 1) does this look useful, particularly for fileserver implementors?
>> >
>> > Yes from the Samba perspective. We'll have to keep the old
>> > code around for compatibility with non-Linux OS'es, but this
>> > will allow Linux Samba to short-circuit a bunch of logic
>> > we have to get around the insane POSIX locking semantics
>> > on close.
>> >
>> > Jeremy.
>>
>> From the peanut gallery, IIRC from college a few years back, wasn't the POSIX file locking stuff passed by all parties because they intended to do their own thing regardless of the standard?  The reason that all locks are blown on a release is mostly because there were already implementations and no one wanted to push the issue, or am I misunderstanding/forgetting the history of file locks in POSIX?
>
> This blog post of Jeremy's explains some of the history:
>
>     http://www.samba.org/samba/news/articles/low_point/tale_two_stds_os2.html
>
> See the section entitled "First Implementation Past the Post".
>
> --
> Jeff Layton <jlayton@redhat.com>

Thanks, Jeff.  That was actually the exact article I was referencing
but forgetting the details of.  Jeremy, thanks for writing that up so
many years ago (I used to eat that stuff up in college).


-- 
Peace and Blessings,
-Scott.

  parent reply	other threads:[~2013-10-12 20:56 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-11 12:25 [RFC PATCH 0/5] locks: implement "filp-private" (aka UNPOSIX) locks Jeff Layton
2013-10-11 12:25 ` [RFC PATCH 1/5] locks: consolidate checks for compatible filp->f_mode values in setlk handlers Jeff Layton
2013-10-11 12:25 ` [RFC PATCH 2/5] locks: add definitions for F_RDLCKP and F_WRLCKP Jeff Layton
2013-10-11 12:25 ` [RFC PATCH 3/5] locks: skip FL_FILP_PRIVATE locks on close unless we're closing the correct filp Jeff Layton
2013-10-11 12:25 ` [RFC PATCH 4/5] locks: handle merging of locks when FL_FILP_PRIVATE is set Jeff Layton
2013-10-11 12:25 ` [RFC PATCH 5/5] locks: show private lock types in /proc/locks Jeff Layton
2013-10-11 13:35 ` [RFC PATCH 0/5] locks: implement "filp-private" (aka UNPOSIX) locks Jeff Layton
2013-10-11 15:20   ` Frank Filz
2013-10-11 15:50     ` Jeff Layton
2013-10-11 17:07       ` Frank Filz
2013-10-11 18:42         ` Jeff Layton
2013-10-11 18:53           ` Frank Filz
2013-10-12  9:10             ` Volker Lendecke
2013-10-11 20:07 ` J. Bruce Fields
2013-10-11 21:36   ` Andreas Dilger
2013-10-11 23:21     ` Jeff Layton
2013-10-11 23:49       ` Jeremy Allison
2013-10-12  0:18         ` Scott Lovenberg
2013-10-12  0:42           ` Jeff Layton
2013-10-12 18:12             ` Frank Filz
2013-10-14  7:24               ` Volker Lendecke
2013-10-14 15:23                 ` Frank Filz
2013-10-15  8:56                   ` Volker Lendecke
2013-10-12 20:56             ` Scott Lovenberg [this message]
2013-10-12  9:20 ` Stefan (metze) Metzmacher
2013-10-12 11:47   ` Jeff Layton
2013-10-12 18:10     ` [Nfs-ganesha-devel] " Frank Filz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAFB9KM2avLpWaMhzZVnB0oXXzWb0NSOBB6oR+RE9JPeuustmvA@mail.gmail.com \
    --to=scott.lovenberg@gmail.com \
    --cc=adilger@dilger.ca \
    --cc=jlayton@redhat.com \
    --cc=jra@samba.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nfs-ganesha-devel@lists.sourceforge.net \
    --cc=samba-technical@lists.samba.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).