linux-kernel - Re: [PATCH v2] staging: lustre: mdc: manage number of modify RPCs in flight

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20161107103832.GA19871@kroah.com>
Date:   Mon, 7 Nov 2016 11:38:32 +0100
From:   Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:     James Simmons <jsimmons@...radead.org>
Cc:     devel@...verdev.osuosl.org,
        Andreas Dilger <andreas.dilger@...el.com>,
        Oleg Drokin <oleg.drokin@...el.com>,
        Gregoire Pichon <gregoire.pichon@...l.net>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Lustre Development List <lustre-devel@...ts.lustre.org>
Subject: Re: [PATCH v2] staging: lustre: mdc: manage number of modify RPCs in
 flight

On Wed, Nov 02, 2016 at 02:46:52PM -0400, James Simmons wrote:
> From: Gregoire Pichon <gregoire.pichon@...l.net>
> 
> This patch is the main client part of a new feature that supports
> multiple modify metadata RPCs in parallel. Its goal is to improve
> metadata operations performance of a single client, while maintening
> the consistency of MDT reply reconstruction and MDT recovery
> mechanisms.
> 
> It allows to manage the number of modify RPCs in flight within
> the client obd structure and to assign a virtual index (the tag) to
> each modify RPC to help server side cleaning of reply data.
> 
> The mdc component uses this feature to send multiple modify RPCs
> in parallel.
> 
> Signed-off-by: Gregoire Pichon <gregoire.pichon@...l.net>
> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5319
> Reviewed-on: http://review.whamcloud.com/14374
> Reviewed-by: Andreas Dilger <andreas.dilger@...el.com>
> Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@...el.com>
> Reviewed-by: Oleg Drokin <oleg.drokin@...el.com>
> Signed-off-by: James Simmons <jsimmons@...radead.org>
> ---
>  drivers/staging/lustre/lustre/fid/fid_request.c    |    4 -
>  drivers/staging/lustre/lustre/include/lustre_mdc.h |   24 +++
>  drivers/staging/lustre/lustre/include/obd.h        |    7 +-
>  drivers/staging/lustre/lustre/include/obd_class.h  |    6 +
>  drivers/staging/lustre/lustre/ldlm/ldlm_lib.c      |   37 +++++
>  drivers/staging/lustre/lustre/mdc/lproc_mdc.c      |   23 +++
>  drivers/staging/lustre/lustre/mdc/mdc_locks.c      |   13 +-
>  drivers/staging/lustre/lustre/mdc/mdc_reint.c      |   23 +--
>  drivers/staging/lustre/lustre/mdc/mdc_request.c    |   50 ++-----
>  drivers/staging/lustre/lustre/obdclass/genops.c    |  171 +++++++++++++++++++-
>  10 files changed, 293 insertions(+), 65 deletions(-)

What changed from v1 of this patch?  You always need to put that below
the --- line so we can remember what is going on here...

Can you make a v3 with that information please?

thanks,

greg k-h