[PATCH 00/22] staging: lustre: Update structs

Peng, Tao tao.peng at emc.com
Mon Oct 21 03:07:07 UTC 2013


Hi Lisa,

Thanks for the patch. I assume you were using coccinelle (or sed) to do the conversion. Are there change done by hand other than scripting? Can you please also put your scripts (such as in [PATCH 01/22]) in the commit message?

Thanks,
Tao

>-----Original Message-----
>From: Lisa Nguyen [mailto:lisa at xenapiadmin.com]
>Sent: Monday, October 21, 2013 6:45 AM
>To: gregkh at linuxfoundation.org
>Cc: Peng, Tao; andreas.dilger at intel.com; devel at driverdev.osuosl.org
>Subject: [PATCH 00/22] staging: lustre: Update structs
>
>This patchset includes all the changes needed to update the structs defined in
>libcfs_hash.h. Doing this will eliminate the "Do not add new typedefs" warning
>generated by checkpatch.pl and the updated format of those structs will meet
>kernel coding style.
>
>Other header files and certain source files that use functions and structs
>defined in libcfs_hash.h have updated struct variables.
>
>I have CC'ed the lustre driver authors to keep them informed of this update.
>
>Questions and comments are welcomed.
>
>Lisa Nguyen (22):
>  staging: lustre: Remove all new typedefs in libcfs_hash.h
>  staging: lustre: Update struct variables in lu_object.h
>  staging: lustre: Update struct variables in lustre_dlm.h
>  staging: lustre: Update struct variables in lustre_export.h
>  staging: lustre: Update struct variables in lustre_net.h
>  staging: lustre: Update struct variables in obd.h
>  staging: lustre: Update struct variables in lov_internal.h
>  staging: lustre: Update struct variables in hash.c
>  staging: lustre: Update struct variables in ldlm_flock.c
>  staging: lustre: Update struct variables in ldlm_lock.c
>  staging: lustre: Update struct variables in ldlm_request.c
>  staging: lustre: Update struct variables in ldlm_resource.c
>  staging: lustre: Update struct variables in ldlm_lockd.c
>  staging: lustre: Update struct variables in vvp_dev.c
>  staging: lustre: Update struct variables in lov_pool.c
>  staging: lustre: Update struct variables in cl_object.c
>  staging: lustre: Update struct variables in genops.c
>  staging: lustre: Update struct variables in lprocfs_status.c
>  staging: lustre: Update struct variables in lu_object.c
>  staging: lustre: Update struct variables in obd_config.c
>  staging: lustre: Update struct variables in osc_quota.c
>  staging: lustre: Update struct variables in connection.c
>
> .../lustre/include/linux/libcfs/libcfs_hash.h      | 280 ++++++++++-----------
> drivers/staging/lustre/lustre/include/lu_object.h  |   4 +-
> drivers/staging/lustre/lustre/include/lustre_dlm.h |   2 +-
> .../staging/lustre/lustre/include/lustre_export.h  |   4 +-
> drivers/staging/lustre/lustre/include/lustre_net.h |   4 +-
> drivers/staging/lustre/lustre/include/obd.h        |  10 +-
> drivers/staging/lustre/lustre/ldlm/ldlm_flock.c    |   8 +-
> drivers/staging/lustre/lustre/ldlm/ldlm_lock.c     |   4 +-
> drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c    |   8 +-
> drivers/staging/lustre/lustre/ldlm/ldlm_request.c  |   4 +-
> drivers/staging/lustre/lustre/ldlm/ldlm_resource.c |  34 +--
> drivers/staging/lustre/lustre/libcfs/hash.c        | 270 ++++++++++----------
> drivers/staging/lustre/lustre/llite/vvp_dev.c      |   2 +-
> drivers/staging/lustre/lustre/lov/lov_internal.h   |   2 +-
> drivers/staging/lustre/lustre/lov/lov_pool.c       |   8 +-
> drivers/staging/lustre/lustre/obdclass/cl_object.c |   8 +-
> drivers/staging/lustre/lustre/obdclass/genops.c    |   6 +-
> .../lustre/lustre/obdclass/lprocfs_status.c        |   6 +-
> drivers/staging/lustre/lustre/obdclass/lu_object.c |  42 ++--
> .../staging/lustre/lustre/obdclass/obd_config.c    |  28 +--
> drivers/staging/lustre/lustre/osc/osc_quota.c      |  10 +-
> drivers/staging/lustre/lustre/ptlrpc/connection.c  |  14 +-
> 22 files changed, 379 insertions(+), 379 deletions(-)
>
>--
>1.8.1.2
>



More information about the devel mailing list