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

Greg Kroah-Hartman gregkh at linuxfoundation.org
Mon Nov 7 02:38:32 PST 2016


On Wed, Nov 02, 2016 at 02:46:52PM -0400, James Simmons wrote:
> From: Gregoire Pichon <gregoire.pichon at bull.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 at bull.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 at intel.com>
> Reviewed-by: Alex Zhuravlev <alexey.zhuravlev at intel.com>
> Reviewed-by: Oleg Drokin <oleg.drokin at intel.com>
> Signed-off-by: James Simmons <jsimmons at infradead.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


More information about the lustre-devel mailing list