Quantcast

[PATCH] cifs: cifsacl: Use a temporary ops variable to reduce code length

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[PATCH] cifs: cifsacl: Use a temporary ops variable to reduce code length

Samba - samba-technical mailing list
Create an ops variable to store tcon->ses->server->ops and cache
indirections and reduce code size a trivial bit.

$ size fs/cifs/cifsacl.o*
   text   data    bss    dec    hex filename
   5338    136      8   5482   156a fs/cifs/cifsacl.o.new
   5371    136      8   5515   158b fs/cifs/cifsacl.o.old

Signed-off-by: Joe Perches <[hidden email]>
---
 fs/cifs/cifsacl.c | 30 ++++++++++++++----------------
 1 file changed, 14 insertions(+), 16 deletions(-)

diff --git a/fs/cifs/cifsacl.c b/fs/cifs/cifsacl.c
index 15bac390dff9..b98436f5c7c7 100644
--- a/fs/cifs/cifsacl.c
+++ b/fs/cifs/cifsacl.c
@@ -1135,20 +1135,19 @@ cifs_acl_to_fattr(struct cifs_sb_info *cifs_sb, struct cifs_fattr *fattr,
  u32 acllen = 0;
  int rc = 0;
  struct tcon_link *tlink = cifs_sb_tlink(cifs_sb);
- struct cifs_tcon *tcon;
+ struct smb_version_operations *ops;
 
  cifs_dbg(NOISY, "converting ACL to mode for %s\n", path);
 
  if (IS_ERR(tlink))
  return PTR_ERR(tlink);
- tcon = tlink_tcon(tlink);
 
- if (pfid && (tcon->ses->server->ops->get_acl_by_fid))
- pntsd = tcon->ses->server->ops->get_acl_by_fid(cifs_sb, pfid,
-  &acllen);
- else if (tcon->ses->server->ops->get_acl)
- pntsd = tcon->ses->server->ops->get_acl(cifs_sb, inode, path,
- &acllen);
+ ops = tlink_tcon(tlink)->ses->server->ops;
+
+ if (pfid && (ops->get_acl_by_fid))
+ pntsd = ops->get_acl_by_fid(cifs_sb, pfid, &acllen);
+ else if (ops->get_acl)
+ pntsd = ops->get_acl(cifs_sb, inode, path, &acllen);
  else {
  cifs_put_tlink(tlink);
  return -EOPNOTSUPP;
@@ -1181,23 +1180,23 @@ id_mode_to_cifs_acl(struct inode *inode, const char *path, __u64 nmode,
  struct cifs_ntsd *pnntsd = NULL; /* modified acl to be sent to server */
  struct cifs_sb_info *cifs_sb = CIFS_SB(inode->i_sb);
  struct tcon_link *tlink = cifs_sb_tlink(cifs_sb);
- struct cifs_tcon *tcon;
+ struct smb_version_operations *ops;
 
  if (IS_ERR(tlink))
  return PTR_ERR(tlink);
- tcon = tlink_tcon(tlink);
+
+ ops = tlink_tcon(tlink)->ses->server->ops;
 
  cifs_dbg(NOISY, "set ACL from mode for %s\n", path);
 
  /* Get the security descriptor */
 
- if (tcon->ses->server->ops->get_acl == NULL) {
+ if (ops->get_acl == NULL) {
  cifs_put_tlink(tlink);
  return -EOPNOTSUPP;
  }
 
- pntsd = tcon->ses->server->ops->get_acl(cifs_sb, inode, path,
- &secdesclen);
+ pntsd = ops->get_acl(cifs_sb, inode, path, &secdesclen);
  if (IS_ERR(pntsd)) {
  rc = PTR_ERR(pntsd);
  cifs_dbg(VFS, "%s: error %d getting sec desc\n", __func__, rc);
@@ -1224,13 +1223,12 @@ id_mode_to_cifs_acl(struct inode *inode, const char *path, __u64 nmode,
 
  cifs_dbg(NOISY, "build_sec_desc rc: %d\n", rc);
 
- if (tcon->ses->server->ops->set_acl == NULL)
+ if (ops->set_acl == NULL)
  rc = -EOPNOTSUPP;
 
  if (!rc) {
  /* Set the security descriptor */
- rc = tcon->ses->server->ops->set_acl(pnntsd, secdesclen, inode,
-     path, aclflag);
+ rc = ops->set_acl(pnntsd, secdesclen, inode, path, aclflag);
  cifs_dbg(NOISY, "set_cifs_acl rc: %d\n", rc);
  }
  cifs_put_tlink(tlink);
--
2.10.0.rc2.1.g053435c


Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [PATCH] cifs: cifsacl: Use a temporary ops variable to reduce code length

Samba - samba-technical mailing list
Looks correct.

Acked-by: Shirish Pargaonkar <[hidden email]>

On Sun, May 7, 2017 at 3:31 AM, Joe Perches via samba-technical
<[hidden email]> wrote:

> Create an ops variable to store tcon->ses->server->ops and cache
> indirections and reduce code size a trivial bit.
>
> $ size fs/cifs/cifsacl.o*
>    text    data     bss     dec     hex filename
>    5338     136       8    5482    156a fs/cifs/cifsacl.o.new
>    5371     136       8    5515    158b fs/cifs/cifsacl.o.old
>
> Signed-off-by: Joe Perches <[hidden email]>
> ---
>  fs/cifs/cifsacl.c | 30 ++++++++++++++----------------
>  1 file changed, 14 insertions(+), 16 deletions(-)
>
> diff --git a/fs/cifs/cifsacl.c b/fs/cifs/cifsacl.c
> index 15bac390dff9..b98436f5c7c7 100644
> --- a/fs/cifs/cifsacl.c
> +++ b/fs/cifs/cifsacl.c
> @@ -1135,20 +1135,19 @@ cifs_acl_to_fattr(struct cifs_sb_info *cifs_sb, struct cifs_fattr *fattr,
>         u32 acllen = 0;
>         int rc = 0;
>         struct tcon_link *tlink = cifs_sb_tlink(cifs_sb);
> -       struct cifs_tcon *tcon;
> +       struct smb_version_operations *ops;
>
>         cifs_dbg(NOISY, "converting ACL to mode for %s\n", path);
>
>         if (IS_ERR(tlink))
>                 return PTR_ERR(tlink);
> -       tcon = tlink_tcon(tlink);
>
> -       if (pfid && (tcon->ses->server->ops->get_acl_by_fid))
> -               pntsd = tcon->ses->server->ops->get_acl_by_fid(cifs_sb, pfid,
> -                                                         &acllen);
> -       else if (tcon->ses->server->ops->get_acl)
> -               pntsd = tcon->ses->server->ops->get_acl(cifs_sb, inode, path,
> -                                                       &acllen);
> +       ops = tlink_tcon(tlink)->ses->server->ops;
> +
> +       if (pfid && (ops->get_acl_by_fid))
> +               pntsd = ops->get_acl_by_fid(cifs_sb, pfid, &acllen);
> +       else if (ops->get_acl)
> +               pntsd = ops->get_acl(cifs_sb, inode, path, &acllen);
>         else {
>                 cifs_put_tlink(tlink);
>                 return -EOPNOTSUPP;
> @@ -1181,23 +1180,23 @@ id_mode_to_cifs_acl(struct inode *inode, const char *path, __u64 nmode,
>         struct cifs_ntsd *pnntsd = NULL; /* modified acl to be sent to server */
>         struct cifs_sb_info *cifs_sb = CIFS_SB(inode->i_sb);
>         struct tcon_link *tlink = cifs_sb_tlink(cifs_sb);
> -       struct cifs_tcon *tcon;
> +       struct smb_version_operations *ops;
>
>         if (IS_ERR(tlink))
>                 return PTR_ERR(tlink);
> -       tcon = tlink_tcon(tlink);
> +
> +       ops = tlink_tcon(tlink)->ses->server->ops;
>
>         cifs_dbg(NOISY, "set ACL from mode for %s\n", path);
>
>         /* Get the security descriptor */
>
> -       if (tcon->ses->server->ops->get_acl == NULL) {
> +       if (ops->get_acl == NULL) {
>                 cifs_put_tlink(tlink);
>                 return -EOPNOTSUPP;
>         }
>
> -       pntsd = tcon->ses->server->ops->get_acl(cifs_sb, inode, path,
> -                                               &secdesclen);
> +       pntsd = ops->get_acl(cifs_sb, inode, path, &secdesclen);
>         if (IS_ERR(pntsd)) {
>                 rc = PTR_ERR(pntsd);
>                 cifs_dbg(VFS, "%s: error %d getting sec desc\n", __func__, rc);
> @@ -1224,13 +1223,12 @@ id_mode_to_cifs_acl(struct inode *inode, const char *path, __u64 nmode,
>
>         cifs_dbg(NOISY, "build_sec_desc rc: %d\n", rc);
>
> -       if (tcon->ses->server->ops->set_acl == NULL)
> +       if (ops->set_acl == NULL)
>                 rc = -EOPNOTSUPP;
>
>         if (!rc) {
>                 /* Set the security descriptor */
> -               rc = tcon->ses->server->ops->set_acl(pnntsd, secdesclen, inode,
> -                                                    path, aclflag);
> +               rc = ops->set_acl(pnntsd, secdesclen, inode, path, aclflag);
>                 cifs_dbg(NOISY, "set_cifs_acl rc: %d\n", rc);
>         }
>         cifs_put_tlink(tlink);
> --
> 2.10.0.rc2.1.g053435c
>
>

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [PATCH] cifs: cifsacl: Use a temporary ops variable to reduce code length

Samba - samba-technical mailing list
merged into cifs-2.6.git for-next

On Tue, May 9, 2017 at 11:26 PM, Shirish Pargaonkar
<[hidden email]> wrote:

> Looks correct.
>
> Acked-by: Shirish Pargaonkar <[hidden email]>
>
> On Sun, May 7, 2017 at 3:31 AM, Joe Perches via samba-technical
> <[hidden email]> wrote:
>> Create an ops variable to store tcon->ses->server->ops and cache
>> indirections and reduce code size a trivial bit.
>>
>> $ size fs/cifs/cifsacl.o*
>>    text    data     bss     dec     hex filename
>>    5338     136       8    5482    156a fs/cifs/cifsacl.o.new
>>    5371     136       8    5515    158b fs/cifs/cifsacl.o.old
>>
>> Signed-off-by: Joe Perches <[hidden email]>
>> ---
>>  fs/cifs/cifsacl.c | 30 ++++++++++++++----------------
>>  1 file changed, 14 insertions(+), 16 deletions(-)
>>
>> diff --git a/fs/cifs/cifsacl.c b/fs/cifs/cifsacl.c
>> index 15bac390dff9..b98436f5c7c7 100644
>> --- a/fs/cifs/cifsacl.c
>> +++ b/fs/cifs/cifsacl.c
>> @@ -1135,20 +1135,19 @@ cifs_acl_to_fattr(struct cifs_sb_info *cifs_sb, struct cifs_fattr *fattr,
>>         u32 acllen = 0;
>>         int rc = 0;
>>         struct tcon_link *tlink = cifs_sb_tlink(cifs_sb);
>> -       struct cifs_tcon *tcon;
>> +       struct smb_version_operations *ops;
>>
>>         cifs_dbg(NOISY, "converting ACL to mode for %s\n", path);
>>
>>         if (IS_ERR(tlink))
>>                 return PTR_ERR(tlink);
>> -       tcon = tlink_tcon(tlink);
>>
>> -       if (pfid && (tcon->ses->server->ops->get_acl_by_fid))
>> -               pntsd = tcon->ses->server->ops->get_acl_by_fid(cifs_sb, pfid,
>> -                                                         &acllen);
>> -       else if (tcon->ses->server->ops->get_acl)
>> -               pntsd = tcon->ses->server->ops->get_acl(cifs_sb, inode, path,
>> -                                                       &acllen);
>> +       ops = tlink_tcon(tlink)->ses->server->ops;
>> +
>> +       if (pfid && (ops->get_acl_by_fid))
>> +               pntsd = ops->get_acl_by_fid(cifs_sb, pfid, &acllen);
>> +       else if (ops->get_acl)
>> +               pntsd = ops->get_acl(cifs_sb, inode, path, &acllen);
>>         else {
>>                 cifs_put_tlink(tlink);
>>                 return -EOPNOTSUPP;
>> @@ -1181,23 +1180,23 @@ id_mode_to_cifs_acl(struct inode *inode, const char *path, __u64 nmode,
>>         struct cifs_ntsd *pnntsd = NULL; /* modified acl to be sent to server */
>>         struct cifs_sb_info *cifs_sb = CIFS_SB(inode->i_sb);
>>         struct tcon_link *tlink = cifs_sb_tlink(cifs_sb);
>> -       struct cifs_tcon *tcon;
>> +       struct smb_version_operations *ops;
>>
>>         if (IS_ERR(tlink))
>>                 return PTR_ERR(tlink);
>> -       tcon = tlink_tcon(tlink);
>> +
>> +       ops = tlink_tcon(tlink)->ses->server->ops;
>>
>>         cifs_dbg(NOISY, "set ACL from mode for %s\n", path);
>>
>>         /* Get the security descriptor */
>>
>> -       if (tcon->ses->server->ops->get_acl == NULL) {
>> +       if (ops->get_acl == NULL) {
>>                 cifs_put_tlink(tlink);
>>                 return -EOPNOTSUPP;
>>         }
>>
>> -       pntsd = tcon->ses->server->ops->get_acl(cifs_sb, inode, path,
>> -                                               &secdesclen);
>> +       pntsd = ops->get_acl(cifs_sb, inode, path, &secdesclen);
>>         if (IS_ERR(pntsd)) {
>>                 rc = PTR_ERR(pntsd);
>>                 cifs_dbg(VFS, "%s: error %d getting sec desc\n", __func__, rc);
>> @@ -1224,13 +1223,12 @@ id_mode_to_cifs_acl(struct inode *inode, const char *path, __u64 nmode,
>>
>>         cifs_dbg(NOISY, "build_sec_desc rc: %d\n", rc);
>>
>> -       if (tcon->ses->server->ops->set_acl == NULL)
>> +       if (ops->set_acl == NULL)
>>                 rc = -EOPNOTSUPP;
>>
>>         if (!rc) {
>>                 /* Set the security descriptor */
>> -               rc = tcon->ses->server->ops->set_acl(pnntsd, secdesclen, inode,
>> -                                                    path, aclflag);
>> +               rc = ops->set_acl(pnntsd, secdesclen, inode, path, aclflag);
>>                 cifs_dbg(NOISY, "set_cifs_acl rc: %d\n", rc);
>>         }
>>         cifs_put_tlink(tlink);
>> --
>> 2.10.0.rc2.1.g053435c
>>
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
> the body of a message to [hidden email]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



--
Thanks,

Steve

Loading...