Linux CIFS filesystem development
 help / color / mirror / Atom feed
From: Jeff Layton <jlayton-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>
To: Steve French <smfrench-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Pavel Shilovsky <piastry-7qunaywFIewox3rIn2DAYQ@public.gmane.org>,
	linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Steve French <sfrench-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
Subject: Re: [PATCH v2 02/53] CIFS: Allow SMB2 statistics to be tracked
Date: Sat, 29 Oct 2011 01:32:05 -0400	[thread overview]
Message-ID: <20111029013205.33647544@corrin.poochiereds.net> (raw)
In-Reply-To: <CAH2r5mtpckA75LEAynsPLVsPfr7U0wnYAyZ1uKeK2VU9F1CGeQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

On Sat, 29 Oct 2011 00:12:45 -0500
Steve French <smfrench-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:

> On Fri, Oct 28, 2011 at 11:48 PM, Jeff Layton <jlayton-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org> wrote:
> > On Fri, 28 Oct 2011 23:54:13 +0400
> > Pavel Shilovsky <piastry-7qunaywFIewox3rIn2DAYQ@public.gmane.org> wrote:
> >
> >> From: Steve French <sfrench-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
> >>
> >> Adding SMB2 statistics requires changes to the way cifs handles stats.
> >> Since there are only 19 command codes, it also is easier to track by exact
> >> command code than it was for cifs.  Turn the counters for protocol
> >> ops sent to be a union (one struct for cifs, one for smb2).  While at it
> >> split out the functions which clear stats and prints stats into their own
> >> subfunctions so they are easy to read and don't go past 80 columns.
> >>
> >> Signed-off-by: Steve French <sfrench-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
> >> Signed-off-by: Pavel Shilovsky <piastry-7qunaywFIewox3rIn2DAYQ@public.gmane.org>
> >> ---
> >>  fs/cifs/cifs_debug.c |  146 ++++++++++++++++++++++++++++++-------------------
> >>  fs/cifs/cifsglob.h   |   56 ++++++++++++-------
> >>  fs/cifs/cifssmb.c    |   54 +++++++++---------
> >>  fs/cifs/misc.c       |    2 +-
> >>  4 files changed, 152 insertions(+), 106 deletions(-)
> >>
> >> diff --git a/fs/cifs/cifs_debug.c b/fs/cifs/cifs_debug.c
> >> index 84e8c07..8ccdb15 100644
> >> --- a/fs/cifs/cifs_debug.c
> >> +++ b/fs/cifs/cifs_debug.c
> >> @@ -249,6 +249,55 @@ static const struct file_operations cifs_debug_data_proc_fops = {
> >>  };
> >>
> >>  #ifdef CONFIG_CIFS_STATS
> >> +
> >> +#ifdef CONFIG_CIFS_SMB2
> >> +static void smb2_clear_stats(struct cifs_tcon *tcon)
> >> +{
> >> +     int i;
> >> +
> >> +     atomic_set(&tcon->num_smbs_sent, 0);
> >> +     for (i = 0; i < NUMBER_OF_SMB2_COMMANDS; i++) {
> >> +             atomic_set(&tcon->stats.smb2_stats.smb2_com_sent[i], 0);
> >> +             atomic_set(&tcon->stats.smb2_stats.smb2_com_fail[i], 0);
> >> +     }
> >> +}
> >> +#endif /* CONFIG_CIFS_SMB2 */
> >> +
> >> +static void clear_cifs_stats(struct cifs_tcon *tcon)
> >> +{
> >> +     atomic_set(&tcon->num_smbs_sent, 0);
> >> +
> >> +#ifdef CONFIG_CIFS_SMB2
> >> +     if (tcon->ses->server->is_smb2) {
> >> +             smb2_clear_stats(tcon);
> >> +             return;
> >> +     }
> >> +#endif /* CONFIG_CIFS_SMB2 */
> >> +
> >
> >     ^^^^^^^^^^^^^
> > The above logic should be encapsulated in smb2_clear_stats, and that
> > function should just be a noop when CONFIG_CIFS_SMB2 is not set.
> 
> makes sense - could be easier to read your way
> 
> 
> >> +     /* cifs specific statistics, not applicable to smb2 sessions */
> >> +     atomic_set(&tcon->stats.cifs_stats.num_writes, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_reads, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_flushes, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_oplock_brks, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_opens, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_posixopens, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_posixmkdirs, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_closes, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_deletes, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_mkdirs, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_rmdirs, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_renames, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_t2renames, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_ffirst, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_fnext, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_fclose, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_hardlinks, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_symlinks, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_locks, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_acl_get, 0);
> >> +     atomic_set(&tcon->stats.cifs_stats.num_acl_set, 0);
> >> +}
> >> +
> >>  static ssize_t cifs_stats_proc_write(struct file *file,
> >>               const char __user *buffer, size_t count, loff_t *ppos)
> >>  {
> >> @@ -279,25 +328,7 @@ static ssize_t cifs_stats_proc_write(struct file *file,
> >>                                       tcon = list_entry(tmp3,
> >>                                                         struct cifs_tcon,
> >>                                                         tcon_list);
> >> -                                     atomic_set(&tcon->num_smbs_sent, 0);
> >> -                                     atomic_set(&tcon->num_writes, 0);
> >> -                                     atomic_set(&tcon->num_reads, 0);
> >> -                                     atomic_set(&tcon->num_oplock_brks, 0);
> >> -                                     atomic_set(&tcon->num_opens, 0);
> >> -                                     atomic_set(&tcon->num_posixopens, 0);
> >> -                                     atomic_set(&tcon->num_posixmkdirs, 0);
> >> -                                     atomic_set(&tcon->num_closes, 0);
> >> -                                     atomic_set(&tcon->num_deletes, 0);
> >> -                                     atomic_set(&tcon->num_mkdirs, 0);
> >> -                                     atomic_set(&tcon->num_rmdirs, 0);
> >> -                                     atomic_set(&tcon->num_renames, 0);
> >> -                                     atomic_set(&tcon->num_t2renames, 0);
> >> -                                     atomic_set(&tcon->num_ffirst, 0);
> >> -                                     atomic_set(&tcon->num_fnext, 0);
> >> -                                     atomic_set(&tcon->num_fclose, 0);
> >> -                                     atomic_set(&tcon->num_hardlinks, 0);
> >> -                                     atomic_set(&tcon->num_symlinks, 0);
> >> -                                     atomic_set(&tcon->num_locks, 0);
> >> +                                     clear_cifs_stats(tcon);
> >>                               }
> >>                       }
> >>               }
> >> @@ -307,6 +338,44 @@ static ssize_t cifs_stats_proc_write(struct file *file,
> >>       return count;
> >>  }
> >>
> >> +static void cifs_stats_print(struct seq_file *m, struct cifs_tcon *tcon)
> >> +{
> >> +     if (tcon->need_reconnect)
> >> +             seq_puts(m, "\tDISCONNECTED ");
> >> +     seq_printf(m, "\nSMBs: %d Oplock Breaks: %d",
> >> +             atomic_read(&tcon->num_smbs_sent),
> >> +             atomic_read(&tcon->stats.cifs_stats.num_oplock_brks));
> >> +     seq_printf(m, "\nReads:  %d Bytes: %lld",
> >> +             atomic_read(&tcon->stats.cifs_stats.num_reads),
> >> +             (long long)(tcon->bytes_read));
> >> +     seq_printf(m, "\nWrites: %d Bytes: %lld",
> >> +             atomic_read(&tcon->stats.cifs_stats.num_writes),
> >> +             (long long)(tcon->bytes_written));
> >> +     seq_printf(m, "\nFlushes: %d",
> >> +             atomic_read(&tcon->stats.cifs_stats.num_flushes));
> >> +     seq_printf(m, "\nLocks: %d HardLinks: %d Symlinks: %d",
> >> +             atomic_read(&tcon->stats.cifs_stats.num_locks),
> >> +             atomic_read(&tcon->stats.cifs_stats.num_hardlinks),
> >> +             atomic_read(&tcon->stats.cifs_stats.num_symlinks));
> >> +     seq_printf(m, "\nOpens: %d Closes: %d Deletes: %d",
> >> +             atomic_read(&tcon->stats.cifs_stats.num_opens),
> >> +             atomic_read(&tcon->stats.cifs_stats.num_closes),
> >> +             atomic_read(&tcon->stats.cifs_stats.num_deletes));
> >> +     seq_printf(m, "\nPosix Opens: %d Posix Mkdirs: %d",
> >> +             atomic_read(&tcon->stats.cifs_stats.num_posixopens),
> >> +             atomic_read(&tcon->stats.cifs_stats.num_posixmkdirs));
> >> +     seq_printf(m, "\nMkdirs: %d Rmdirs: %d",
> >> +             atomic_read(&tcon->stats.cifs_stats.num_mkdirs),
> >> +             atomic_read(&tcon->stats.cifs_stats.num_rmdirs));
> >> +     seq_printf(m, "\nRenames: %d T2 Renames %d",
> >> +             atomic_read(&tcon->stats.cifs_stats.num_renames),
> >> +             atomic_read(&tcon->stats.cifs_stats.num_t2renames));
> >> +     seq_printf(m, "\nFindFirst: %d FNext %d FClose %d",
> >> +             atomic_read(&tcon->stats.cifs_stats.num_ffirst),
> >> +             atomic_read(&tcon->stats.cifs_stats.num_fnext),
> >> +             atomic_read(&tcon->stats.cifs_stats.num_fclose));
> >> +}
> >> +
> >>  static int cifs_stats_proc_show(struct seq_file *m, void *v)
> >>  {
> >>       int i;
> >> @@ -354,44 +423,7 @@ static int cifs_stats_proc_show(struct seq_file *m, void *v)
> >>                                                 tcon_list);
> >>                               i++;
> >>                               seq_printf(m, "\n%d) %s", i, tcon->treeName);
> >> -                             if (tcon->need_reconnect)
> >> -                                     seq_puts(m, "\tDISCONNECTED ");
> >> -                             seq_printf(m, "\nSMBs: %d Oplock Breaks: %d",
> >> -                                     atomic_read(&tcon->num_smbs_sent),
> >> -                                     atomic_read(&tcon->num_oplock_brks));
> >> -                             seq_printf(m, "\nReads:  %d Bytes: %lld",
> >> -                                     atomic_read(&tcon->num_reads),
> >> -                                     (long long)(tcon->bytes_read));
> >> -                             seq_printf(m, "\nWrites: %d Bytes: %lld",
> >> -                                     atomic_read(&tcon->num_writes),
> >> -                                     (long long)(tcon->bytes_written));
> >> -                             seq_printf(m, "\nFlushes: %d",
> >> -                                     atomic_read(&tcon->num_flushes));
> >> -                             seq_printf(m, "\nLocks: %d HardLinks: %d "
> >> -                                           "Symlinks: %d",
> >> -                                     atomic_read(&tcon->num_locks),
> >> -                                     atomic_read(&tcon->num_hardlinks),
> >> -                                     atomic_read(&tcon->num_symlinks));
> >> -                             seq_printf(m, "\nOpens: %d Closes: %d "
> >> -                                           "Deletes: %d",
> >> -                                     atomic_read(&tcon->num_opens),
> >> -                                     atomic_read(&tcon->num_closes),
> >> -                                     atomic_read(&tcon->num_deletes));
> >> -                             seq_printf(m, "\nPosix Opens: %d "
> >> -                                           "Posix Mkdirs: %d",
> >> -                                     atomic_read(&tcon->num_posixopens),
> >> -                                     atomic_read(&tcon->num_posixmkdirs));
> >> -                             seq_printf(m, "\nMkdirs: %d Rmdirs: %d",
> >> -                                     atomic_read(&tcon->num_mkdirs),
> >> -                                     atomic_read(&tcon->num_rmdirs));
> >> -                             seq_printf(m, "\nRenames: %d T2 Renames %d",
> >> -                                     atomic_read(&tcon->num_renames),
> >> -                                     atomic_read(&tcon->num_t2renames));
> >> -                             seq_printf(m, "\nFindFirst: %d FNext %d "
> >> -                                           "FClose %d",
> >> -                                     atomic_read(&tcon->num_ffirst),
> >> -                                     atomic_read(&tcon->num_fnext),
> >> -                                     atomic_read(&tcon->num_fclose));
> >> +                             cifs_stats_print(m, tcon);
> >>                       }
> >>               }
> >>       }
> >> diff --git a/fs/cifs/cifsglob.h b/fs/cifs/cifsglob.h
> >> index 4c38b04..6dfc7ef 100644
> >> --- a/fs/cifs/cifsglob.h
> >> +++ b/fs/cifs/cifsglob.h
> >> @@ -292,6 +292,7 @@ struct TCP_Server_Info {
> >>       bool    sec_kerberos;           /* supports plain Kerberos */
> >>       bool    sec_mskerberos;         /* supports legacy MS Kerberos */
> >>       bool    large_buf;              /* is current buffer large? */
> >> +     bool    is_smb2;                /* smb2 not cifs protocol negotiated */
> >>       struct delayed_work     echo; /* echo ping workqueue job */
> >>       struct kvec *iov;       /* reusable kvec array for receives */
> >>       unsigned int nr_iov;    /* number of kvecs in array */
> >> @@ -392,6 +393,9 @@ struct cifs_ses {
> >>     negotiate one of the older LANMAN dialects */
> >>  #define CIFS_SES_LANMAN 8
> >>  #define CIFS_SES_SMB2 16
> >> +
> >> +#define NUMBER_OF_SMB2_COMMANDS 0x0013
> >> +
> >>  /*
> >>   * there is one of these for each connection to a resource on a particular
> >>   * session
> >> @@ -409,27 +413,37 @@ struct cifs_tcon {
> >>       enum statusEnum tidStatus;
> >>  #ifdef CONFIG_CIFS_STATS
> >>       atomic_t num_smbs_sent;
> >> -     atomic_t num_writes;
> >> -     atomic_t num_reads;
> >> -     atomic_t num_flushes;
> >> -     atomic_t num_oplock_brks;
> >> -     atomic_t num_opens;
> >> -     atomic_t num_closes;
> >> -     atomic_t num_deletes;
> >> -     atomic_t num_mkdirs;
> >> -     atomic_t num_posixopens;
> >> -     atomic_t num_posixmkdirs;
> >> -     atomic_t num_rmdirs;
> >> -     atomic_t num_renames;
> >> -     atomic_t num_t2renames;
> >> -     atomic_t num_ffirst;
> >> -     atomic_t num_fnext;
> >> -     atomic_t num_fclose;
> >> -     atomic_t num_hardlinks;
> >> -     atomic_t num_symlinks;
> >> -     atomic_t num_locks;
> >> -     atomic_t num_acl_get;
> >> -     atomic_t num_acl_set;
> >> +     union {
> >> +             struct {
> >> +                     atomic_t num_writes;
> >> +                     atomic_t num_reads;
> >> +                     atomic_t num_flushes;
> >> +                     atomic_t num_oplock_brks;
> >> +                     atomic_t num_opens;
> >> +                     atomic_t num_closes;
> >> +                     atomic_t num_deletes;
> >> +                     atomic_t num_mkdirs;
> >> +                     atomic_t num_posixopens;
> >> +                     atomic_t num_posixmkdirs;
> >> +                     atomic_t num_rmdirs;
> >> +                     atomic_t num_renames;
> >> +                     atomic_t num_t2renames;
> >> +                     atomic_t num_ffirst;
> >> +                     atomic_t num_fnext;
> >> +                     atomic_t num_fclose;
> >> +                     atomic_t num_hardlinks;
> >> +                     atomic_t num_symlinks;
> >> +                     atomic_t num_locks;
> >> +                     atomic_t num_acl_get;
> >> +                     atomic_t num_acl_set;
> >> +             } cifs_stats;
> >> +#ifdef CONFIG_CIFS_SMB2
> >> +             struct {
> >> +                     atomic_t smb2_com_sent[NUMBER_OF_SMB2_COMMANDS];
> >> +                     atomic_t smb2_com_fail[NUMBER_OF_SMB2_COMMANDS];
> >> +             } smb2_stats;
> >
> > Is it really necessary to do this with atomics? Those can have
> > significant performance impact (TLB flushes, and we don't seem to need the
> > guarantees that they provide for simple counters like this. Perhaps
> > these should be switched to per-cpu variables or just plain ints?
> 
> presumably there is precedent for use of atomics rather than
> wrapping updates to these in a mutex - if they are just int
> would we have a case where the two overlapping updates could
> badly corrupt the value?
> 

There's no need for any locking here. I think what you want is a per-cpu
counter for these.

-- 
Jeff Layton <jlayton-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>

  parent reply	other threads:[~2011-10-29  5:32 UTC|newest]

Thread overview: 75+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-28 19:54 [PATCH v2 00/53] SMB2 protocol support for CIFS kernel module Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 02/53] CIFS: Allow SMB2 statistics to be tracked Pavel Shilovsky
2011-10-29  4:48   ` Jeff Layton
2011-10-29  5:12     ` Steve French
     [not found]       ` <CAH2r5mtpckA75LEAynsPLVsPfr7U0wnYAyZ1uKeK2VU9F1CGeQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-10-29  5:32         ` Jeff Layton [this message]
     [not found]     ` <20111029004814.41340503-4QP7MXygkU+dMjc06nkz3ljfA9RmPOcC@public.gmane.org>
2011-10-29 19:44       ` Pavel Shilovsky
     [not found]         ` <CAKywueQwBwR-HZtv6gQv32kt4+2qycS7vq5KDLiE3xw3_9=wPA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-10-30  2:12           ` Jeff Layton
     [not found]             ` <20111029221251.5fba1444-4QP7MXygkU+dMjc06nkz3ljfA9RmPOcC@public.gmane.org>
2011-10-30  2:52               ` Steve French
2011-10-28 19:54 ` [PATCH v2 03/53] CIFS: Check for smb2 vs. cifs in find_tcp_session Pavel Shilovsky
     [not found]   ` <1319831704-3572-4-git-send-email-piastry-7qunaywFIewox3rIn2DAYQ@public.gmane.org>
2011-10-29  4:40     ` Jeff Layton
     [not found]       ` <20111029004046.2bf8e111-4QP7MXygkU+dMjc06nkz3ljfA9RmPOcC@public.gmane.org>
2011-10-29 19:55         ` Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 05/53] CIFS: wait_for_free_request needs to wait on credits returned by server (for SMB2) Pavel Shilovsky
     [not found]   ` <1319831704-3572-6-git-send-email-piastry-7qunaywFIewox3rIn2DAYQ@public.gmane.org>
2011-10-29 15:35     ` Jeff Layton
2011-10-29 20:00       ` Pavel Shilovsky
     [not found]         ` <CAKywueSGDSw33L4_a7W6D7gM_PH9uy3xTQ6=8zgAimGhz5=UfA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-10-30  2:41           ` Jeff Layton
2011-10-28 19:54 ` [PATCH v2 07/53] CIFS: Add structure definitions for SMB2 PDUs Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 09/53] CIFS: Allocating SMB2 mids (multiplex identifier structures) Pavel Shilovsky
     [not found] ` <1319831704-3572-1-git-send-email-piastry-7qunaywFIewox3rIn2DAYQ@public.gmane.org>
2011-10-28 19:54   ` [PATCH v2 01/53] CIFS: Update cifs global structures to handle smb2 sessions Pavel Shilovsky
     [not found]     ` <1319831704-3572-2-git-send-email-piastry-7qunaywFIewox3rIn2DAYQ@public.gmane.org>
2011-10-29  4:54       ` Jeff Layton
2011-10-28 19:54   ` [PATCH v2 04/53] CIFS: Do not try to dump cifs mids from " Pavel Shilovsky
     [not found]     ` <1319831704-3572-5-git-send-email-piastry-7qunaywFIewox3rIn2DAYQ@public.gmane.org>
2011-10-29  5:05       ` Jeff Layton
2011-10-28 19:54   ` [PATCH v2 06/53] CIFS: Add missing unicode handling routines needed by smb2 Pavel Shilovsky
     [not found]     ` <1319831704-3572-7-git-send-email-piastry-7qunaywFIewox3rIn2DAYQ@public.gmane.org>
2011-11-01 10:12       ` Jeff Layton
2011-11-01 13:52       ` Shirish Pargaonkar
2011-12-29 15:01         ` Pavel Shilovsky
2011-12-30 12:42           ` Jeff Layton
2012-01-12 10:47             ` Pavel Shilovsky
2012-01-12 17:22       ` Shirish Pargaonkar
     [not found]         ` <CADT32eJ7FMkQZiy4LjeEG3axvt99N7jFEod7-u2js1MnhVkc7Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-01-12 18:42           ` Jeff Layton
2012-01-12 19:58             ` Shirish Pargaonkar
     [not found]               ` <CADT32eKc=Xq5G1cE+pye6h1dN6sjry+b7V0ze7CTMJXC8JCtwA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-01-12 20:48                 ` Jeff Layton
2012-01-12 20:41       ` Shirish Pargaonkar
2011-10-28 19:54   ` [PATCH v2 10/53] CIFS: Add routines to free SMB2 mids Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 11/53] CIFS: Add sync_smb2_mid_result Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 13/53] CIFS: Add SMB2 transport routines Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 14/53] CIFS: Expand cifs mid structure to keep SMB2 related fields Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 17/53] CIFS: Simplify SMB2 query info Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 20/53] CIFS: Add SMB2 support for rmdir operation Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 21/53] CIFS: Add SMB2 support for unlink operation Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 24/53] CIFS: Add SMB2 support for cifs_iovec_write Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 27/53] CIFS: Add read related address space operations for SMB2 Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 29/53] CIFS: Respect max buf size for SMB2 read and write Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 34/53] [CIFS] Add SMB2 support for cifs_get_file_info Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 35/53] CIFS: Add SMB2 support for create operation Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 36/53] CIFS: Add readdir support for SMB2 Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 37/53] CIFS: Add SMB2 support for rename operation Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 38/53] CIFS: Add SMB2 support for hardlink operation Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 40/53] CIFS: Add NTLMSSP sec type to defaults Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 41/53] CIFS: Disable SMB2.1 protocol negotiating Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 43/53] CIFS: Process STATUS_PENDING responses for SMB2 Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 44/53] CIFS: Request SMB2.1 leases Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 45/53] CIFS: Process oplock/lease break for SMB2/2.1 Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 47/53] CIFS: Enable signing in SMB2 Pavel Shilovsky
2011-10-28 19:54   ` [PATCH v2 48/53] CIFS: Process signing for SMB2_logoff Pavel Shilovsky
2011-10-28 19:55   ` [PATCH v2 49/53] CIFS: Introduce SMB2 Kconfig option Pavel Shilovsky
2011-10-28 19:55   ` [PATCH v2 50/53] CIFS: Introduce smb2 mounts as vers=2 Pavel Shilovsky
2011-10-28 19:55   ` [PATCH v2 53/53] CIFS: Disable lock call for SMB2 since we don't support it Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 15/53] CIFS: Make demultiplex_thread work with SMB2 code Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 16/53] CIFS: Get mount/umount work with SMB2 protocol Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 18/53] CIFS: Add SMB2 inode/dentry ops structures Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 19/53] CIFS: Add SMB2 support for mkdir operation Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 22/53] CIFS: Add SMB2 support for open/close file operations Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 23/53] CIFS: Add SMB2 support for reopen file operation Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 25/53] CIFS: Add SMB2 support for cifs_iovec_read Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 26/53] CIFS: Add address space ops structures for SMB2 Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 28/53] CIFS: Add write related address space operations " Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 30/53] CIFS: Temporarily disable set inode info " Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 31/53] CIFS: Add writepages support " Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 32/53] CIFS: Add readpages " Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 33/53] CIFS: Add echo request " Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 39/53] CIFS: Add SMB2 support for flush operation Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 42/53] CIFS: Process oplocks for SMB2 Pavel Shilovsky
2011-10-28 19:54 ` [PATCH v2 46/53] CIFS: Add strictcache i/o " Pavel Shilovsky
2011-10-28 19:55 ` [PATCH v2 51/53] CIFS: Change Makefile to support CONFIG_CIFS_SMB2 Pavel Shilovsky
2011-10-28 19:55 ` [PATCH v2 52/53] CIFS: Add statfs support for SMB2 Pavel Shilovsky

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=20111029013205.33647544@corrin.poochiereds.net \
    --to=jlayton-eunubhrolfbytjvyw6ydsg@public.gmane.org \
    --cc=linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=piastry-7qunaywFIewox3rIn2DAYQ@public.gmane.org \
    --cc=sfrench-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org \
    --cc=smfrench-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.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