All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andreas Gruenbacher <andreas.gruenbacher@linbit.com>
To: cluster-devel.redhat.com
Subject: [Cluster-devel] [PATCH 02/18] fs: add get_acl helper
Date: Thu, 12 Dec 2013 20:06:09 +0100 (CET)	[thread overview]
Message-ID: <1724169901.69108.1386875169382.JavaMail.zimbra@linbit.com> (raw)
In-Reply-To: <20131211104526.648135334@bombadil.infradead.org>

Christoph,

> +struct posix_acl *get_acl(struct inode *inode, int type)
> +{
> +	struct posix_acl *acl;
> +
> +	acl = get_cached_acl(inode, type);
> +	if (acl != ACL_NOT_CACHED)
> +		return acl;
> +
> +	if (!IS_POSIXACL(inode))
> +		return NULL;
> +
> +	/*
> +	 * A filesystem can force a ACL callback by just never filling the
> +	 * ACL cache. But normally you'd fill the cache either at inode
> +	 * instantiation time, or on the first ->get_acl call.
> +	 *
> +	 * If the filesystem doesn't have a get_acl() function at all, we'll
> +	 * just create the negative cache entry.
> +	 */
> +	if (!inode->i_op->get_acl) {
> +		set_cached_acl(inode, type, NULL);
> +		return ERR_PTR(-EAGAIN);

The function should return NULL here.

Andreas



WARNING: multiple messages have this Message-ID (diff)
From: Andreas Gruenbacher <andreas.gruenbacher@linbit.com>
To: Christoph Hellwig <hch@infradead.org>
Cc: viro@zeniv.linux.org.uk, linux-fsdevel@vger.kernel.org,
	linux-btrfs@vger.kernel.org, linux-ext4@vger.kernel.org,
	linux-f2fs-devel@lists.sourceforge.net,
	linux-mtd@lists.infradead.org, Mark Fasheh <mfasheh@suse.com>,
	Joel Becker <jlbec@evilplan.org>,
	reiserfs-devel@vger.kernel.org, xfs@oss.sgi.com,
	jfs-discussion@lists.sourceforge.net, cluster-devel@redhat.com,
	linux-nfs@vger.kernel.org, Jan Kara <jack@suse.cz>
Subject: Re: [PATCH 02/18] fs: add get_acl helper
Date: Thu, 12 Dec 2013 20:06:09 +0100 (CET)	[thread overview]
Message-ID: <1724169901.69108.1386875169382.JavaMail.zimbra@linbit.com> (raw)
In-Reply-To: <20131211104526.648135334@bombadil.infradead.org>

Christoph,

> +struct posix_acl *get_acl(struct inode *inode, int type)
> +{
> +	struct posix_acl *acl;
> +
> +	acl = get_cached_acl(inode, type);
> +	if (acl != ACL_NOT_CACHED)
> +		return acl;
> +
> +	if (!IS_POSIXACL(inode))
> +		return NULL;
> +
> +	/*
> +	 * A filesystem can force a ACL callback by just never filling the
> +	 * ACL cache. But normally you'd fill the cache either at inode
> +	 * instantiation time, or on the first ->get_acl call.
> +	 *
> +	 * If the filesystem doesn't have a get_acl() function at all, we'll
> +	 * just create the negative cache entry.
> +	 */
> +	if (!inode->i_op->get_acl) {
> +		set_cached_acl(inode, type, NULL);
> +		return ERR_PTR(-EAGAIN);

The function should return NULL here.

Andreas

WARNING: multiple messages have this Message-ID (diff)
From: Andreas Gruenbacher <andreas.gruenbacher@linbit.com>
To: Christoph Hellwig <hch@infradead.org>
Cc: xfs@oss.sgi.com, Jan Kara <jack@suse.cz>,
	Mark Fasheh <mfasheh@suse.com>,
	reiserfs-devel@vger.kernel.org,
	linux-f2fs-devel@lists.sourceforge.net, cluster-devel@redhat.com,
	linux-mtd@lists.infradead.org, viro@zeniv.linux.org.uk,
	jfs-discussion@lists.sourceforge.net,
	linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org,
	linux-nfs@vger.kernel.org, linux-btrfs@vger.kernel.org,
	Joel Becker <jlbec@evilplan.org>
Subject: Re: [PATCH 02/18] fs: add get_acl helper
Date: Thu, 12 Dec 2013 20:06:09 +0100 (CET)	[thread overview]
Message-ID: <1724169901.69108.1386875169382.JavaMail.zimbra@linbit.com> (raw)
In-Reply-To: <20131211104526.648135334@bombadil.infradead.org>

Christoph,

> +struct posix_acl *get_acl(struct inode *inode, int type)
> +{
> +	struct posix_acl *acl;
> +
> +	acl = get_cached_acl(inode, type);
> +	if (acl != ACL_NOT_CACHED)
> +		return acl;
> +
> +	if (!IS_POSIXACL(inode))
> +		return NULL;
> +
> +	/*
> +	 * A filesystem can force a ACL callback by just never filling the
> +	 * ACL cache. But normally you'd fill the cache either at inode
> +	 * instantiation time, or on the first ->get_acl call.
> +	 *
> +	 * If the filesystem doesn't have a get_acl() function at all, we'll
> +	 * just create the negative cache entry.
> +	 */
> +	if (!inode->i_op->get_acl) {
> +		set_cached_acl(inode, type, NULL);
> +		return ERR_PTR(-EAGAIN);

The function should return NULL here.

Andreas

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

WARNING: multiple messages have this Message-ID (diff)
From: Andreas Gruenbacher <andreas.gruenbacher@linbit.com>
To: Christoph Hellwig <hch@infradead.org>
Cc: xfs@oss.sgi.com, Jan Kara <jack@suse.cz>,
	Mark Fasheh <mfasheh@suse.com>,
	reiserfs-devel@vger.kernel.org,
	linux-f2fs-devel@lists.sourceforge.net, cluster-devel@redhat.com,
	linux-mtd@lists.infradead.org, viro@zeniv.linux.org.uk,
	jfs-discussion@lists.sourceforge.net,
	linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org,
	linux-nfs@vger.kernel.org, linux-btrfs@vger.kernel.org,
	Joel Becker <jlbec@evilplan.org>
Subject: Re: [PATCH 02/18] fs: add get_acl helper
Date: Thu, 12 Dec 2013 20:06:09 +0100 (CET)	[thread overview]
Message-ID: <1724169901.69108.1386875169382.JavaMail.zimbra@linbit.com> (raw)
In-Reply-To: <20131211104526.648135334@bombadil.infradead.org>

Christoph,

> +struct posix_acl *get_acl(struct inode *inode, int type)
> +{
> +	struct posix_acl *acl;
> +
> +	acl = get_cached_acl(inode, type);
> +	if (acl != ACL_NOT_CACHED)
> +		return acl;
> +
> +	if (!IS_POSIXACL(inode))
> +		return NULL;
> +
> +	/*
> +	 * A filesystem can force a ACL callback by just never filling the
> +	 * ACL cache. But normally you'd fill the cache either at inode
> +	 * instantiation time, or on the first ->get_acl call.
> +	 *
> +	 * If the filesystem doesn't have a get_acl() function at all, we'll
> +	 * just create the negative cache entry.
> +	 */
> +	if (!inode->i_op->get_acl) {
> +		set_cached_acl(inode, type, NULL);
> +		return ERR_PTR(-EAGAIN);

The function should return NULL here.

Andreas

  reply	other threads:[~2013-12-12 19:06 UTC|newest]

Thread overview: 131+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-11 10:42 [Cluster-devel] [PATCH 00/18] Consolidate Posix ACL implementation V2 Christoph Hellwig
2013-12-11 10:42 ` Christoph Hellwig
2013-12-11 10:42 ` Christoph Hellwig
2013-12-11 10:42 ` Christoph Hellwig
2013-12-11 10:42 ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 01/18] reiserfs: prefix ACL symbols with reiserfs_ Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 02/18] fs: add get_acl helper Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-12 19:06   ` Andreas Gruenbacher [this message]
2013-12-12 19:06     ` Andreas Gruenbacher
2013-12-12 19:06     ` Andreas Gruenbacher
2013-12-12 19:06     ` Andreas Gruenbacher
2013-12-12 21:04     ` [Cluster-devel] " Christoph Hellwig
2013-12-12 21:04       ` Christoph Hellwig
2013-12-12 21:04       ` Christoph Hellwig
2013-12-12 21:04       ` Christoph Hellwig
2013-12-12 21:04       ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 03/18] fs: add a set_acl inode operation Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 04/18] fs: add generic xattr_acl handlers Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-12 19:07   ` [Cluster-devel] " Andreas Gruenbacher
2013-12-12 19:07     ` Andreas Gruenbacher
2013-12-12 19:07     ` Andreas Gruenbacher
2013-12-12 19:07     ` Andreas Gruenbacher
2013-12-12 19:07     ` Andreas Gruenbacher
2013-12-11 10:42 ` [Cluster-devel] [PATCH 05/18] fs: make posix_acl_chmod more useful Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-12 19:07   ` [Cluster-devel] " Andreas Gruenbacher
2013-12-12 19:07     ` Andreas Gruenbacher
2013-12-12 19:07     ` Andreas Gruenbacher
2013-12-12 19:07     ` Andreas Gruenbacher
2013-12-12 21:05     ` [Cluster-devel] " Christoph Hellwig
2013-12-12 21:05       ` Christoph Hellwig
2013-12-12 21:05       ` Christoph Hellwig
2013-12-12 21:05       ` Christoph Hellwig
2013-12-12 21:05       ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 06/18] fs: make posix_acl_create " Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 07/18] btrfs: use generic posix ACL infrastructure Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 08/18] ext2/3/4: " Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 09/18] f2fs: " Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 10/18] hfsplus: " Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 11/18] jffs2: " Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 12/18] ocfs2: " Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 13/18] reiserfs: " Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 14/18] xfs: " Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 15/18] jfs: " Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42 ` [Cluster-devel] [PATCH 16/18] gfs2: " Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:42   ` Christoph Hellwig
2013-12-11 10:52   ` [Cluster-devel] " Steven Whitehouse
2013-12-11 10:52     ` Steven Whitehouse
2013-12-11 10:52     ` Steven Whitehouse
2013-12-11 10:52     ` Steven Whitehouse
2013-12-11 10:52     ` Steven Whitehouse
2013-12-12 19:08   ` Andreas Gruenbacher
2013-12-12 19:08     ` Andreas Gruenbacher
2013-12-12 19:08     ` Andreas Gruenbacher
2013-12-12 19:08     ` Andreas Gruenbacher
2013-12-12 21:05     ` [Cluster-devel] " Christoph Hellwig
2013-12-12 21:05       ` Christoph Hellwig
2013-12-12 21:05       ` Christoph Hellwig
2013-12-12 21:05       ` Christoph Hellwig
2013-12-12 21:05       ` Christoph Hellwig
2013-12-11 10:43 ` [Cluster-devel] [PATCH 17/18] nfs: use generic posix ACL infrastructure for v3 Posix ACLs Christoph Hellwig
2013-12-11 10:43   ` Christoph Hellwig
2013-12-11 10:43   ` Christoph Hellwig
2013-12-11 10:43   ` Christoph Hellwig
2013-12-11 10:43   ` Christoph Hellwig
2013-12-11 10:43 ` [Cluster-devel] [PATCH 18/18] fs: remove generic_acl Christoph Hellwig
2013-12-11 10:43   ` Christoph Hellwig
2013-12-11 10:43   ` Christoph Hellwig
2013-12-11 10:43   ` Christoph Hellwig
2013-12-11 10:43   ` Christoph Hellwig
  -- strict thread matches above, loose matches on Subject: below --
2013-12-01 11:59 [Cluster-devel] [PATCH 00/18] Consolidate Posix ACL implementation Christoph Hellwig
2013-12-01 11:59 ` [Cluster-devel] [PATCH 02/18] fs: add get_acl helper Christoph Hellwig
2013-12-02 20:14   ` Jan Kara

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=1724169901.69108.1386875169382.JavaMail.zimbra@linbit.com \
    --to=andreas.gruenbacher@linbit.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.