diff mbox

cfuse recognizing same xattr subset as kernel client

Message ID BANLkTikZKszEiZqUNygTx+BxbAy8j3b_LQ@mail.gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Brian Chrisman April 8, 2011, 10:42 p.m. UTC
Hopefully this patch is formatted/signed off correctly.

Signed-off-by: Brian Chrisman <brchrisman@gmail.com>
---
 src/client/Client.cc |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)


   Inode *in = _ll_get_inode(vino);
--
1.7.1
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Sage Weil April 8, 2011, 11:23 p.m. UTC | #1
On Fri, 8 Apr 2011, Brian Chrisman wrote:
> Hopefully this patch is formatted/signed off correctly.

Applied, thanks!

I did have to fix the word wrapping.  Depending on your mailer you should 
be able to inline the message without reformatting (^R in pine, if you're 
old like me).

sage


> 
> Signed-off-by: Brian Chrisman <brchrisman@gmail.com>
> ---
>  src/client/Client.cc |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/client/Client.cc b/src/client/Client.cc
> index aa50452..95f102a 100644
> --- a/src/client/Client.cc
> +++ b/src/client/Client.cc
> @@ -5715,8 +5715,8 @@ int Client::ll_setxattr(vinodeno_t vino, const
> char *name, const void *value, si
>    tout << vino.ino.val << std::endl;
>    tout << name << std::endl;
> 
> -  // only user xattrs, for now
> -  if (strncmp(name, "user.", 5))
> +  // same xattrs supported by kernel client
> +  if (strncmp(name, "user.", 5) && strncmp(name, "security.", 9) &&
> strncmp(name, "trusted.", 8))
>      return -EOPNOTSUPP;
> 
>    Inode *in = _ll_get_inode(vino);
> --
> 1.7.1
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/src/client/Client.cc b/src/client/Client.cc
index aa50452..95f102a 100644
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -5715,8 +5715,8 @@  int Client::ll_setxattr(vinodeno_t vino, const
char *name, const void *value, si
   tout << vino.ino.val << std::endl;
   tout << name << std::endl;

-  // only user xattrs, for now
-  if (strncmp(name, "user.", 5))
+  // same xattrs supported by kernel client
+  if (strncmp(name, "user.", 5) && strncmp(name, "security.", 9) &&
strncmp(name, "trusted.", 8))
     return -EOPNOTSUPP;