From patchwork Thu Feb 7 00:03:32 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: NeilBrown X-Patchwork-Id: 10800295 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 6A821746 for ; Thu, 7 Feb 2019 00:05:57 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 54CF52D348 for ; Thu, 7 Feb 2019 00:05:57 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 476302D5C8; Thu, 7 Feb 2019 00:05:57 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.1 Received: from pdx1-mailman02.dreamhost.com (pdx1-mailman02.dreamhost.com [64.90.62.194]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id C94BF2D348 for ; Thu, 7 Feb 2019 00:05:56 +0000 (UTC) Received: from pdx1-mailman02.dreamhost.com (localhost [IPv6:::1]) by pdx1-mailman02.dreamhost.com (Postfix) with ESMTP id EEFEA4C3CA4; Wed, 6 Feb 2019 16:05:55 -0800 (PST) X-Original-To: lustre-devel@lists.lustre.org Delivered-To: lustre-devel-lustre.org@pdx1-mailman02.dreamhost.com Received: from mx1.suse.de (mx2.suse.de [195.135.220.15]) by pdx1-mailman02.dreamhost.com (Postfix) with ESMTP id 411074C3C85 for ; Wed, 6 Feb 2019 16:05:54 -0800 (PST) X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id 16925AE5E; Thu, 7 Feb 2019 00:05:53 +0000 (UTC) From: NeilBrown To: Andreas Dilger , James Simmons , Oleg Drokin Date: Thu, 07 Feb 2019 11:03:32 +1100 Message-ID: <154949776249.10620.1215070753973826063.stgit@noble.brown> User-Agent: StGit/0.17.1-dirty MIME-Version: 1.0 Subject: [lustre-devel] [PATCH 00/21] lustre: Assorted cleanups for obdclass X-BeenThere: lustre-devel@lists.lustre.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "For discussing Lustre software development." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Lustre Development List Errors-To: lustre-devel-bounces@lists.lustre.org Sender: "lustre-devel" X-Virus-Scanned: ClamAV using ClamSMTP I've been reading through obdclass and writing patches as I go. I'm not done yet, but this patch set to big enough to send for review. NeilBrown --- NeilBrown (21): lustre: obdclass: discard csi_end_io lustre: obd_class: remove csi_barrier from struct cl_sync_io lustre: obdclass: use list_sort() to sort a list. lustre: use list*entry macros in place of container_of() lustre: use list_first_entry() in lustre subdirectory. lustre: use list_first_entry() in lnet/lnet subdirectory. lustre: use list_first_entry() in lnet/klnds subdirectory. lustre: use list_first_entry() throughout lustre: use list_last_entry() throughout lustre: obdclass: use cl_object_for_each where appropriate lustre: cl_object: remove vestigial debugging. lustre: cl_page.c: remove PINVRNT() lustre: make cp_ref in cl_page a refcount_t lustre: make ccc_users in cl_client_cache a refcount_t lustre: obdclass: char obd_ioctl_getdata type. lustre: obdclass: normalize a switch statement. lustre: obdclass: result of try_module_get() should not be ignored. lustre: move debug.c from obdclass to obdecho lustre: obdclass: avoid races in class_register_type() lustre: obdclass: fix module load locking. lustre: make exp_refcount in obd_export a refcount_t .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c | 19 ++- .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 61 ++++++----- .../staging/lustre/lnet/klnds/socklnd/socklnd.c | 9 +- .../staging/lustre/lnet/klnds/socklnd/socklnd_cb.c | 79 +++++++------- .../lustre/lnet/klnds/socklnd/socklnd_proto.c | 4 - drivers/staging/lustre/lnet/libcfs/libcfs_string.c | 12 +- drivers/staging/lustre/lnet/lnet/api-ni.c | 111 ++++++++++---------- drivers/staging/lustre/lnet/lnet/config.c | 22 ++-- drivers/staging/lustre/lnet/lnet/lib-move.c | 53 +++++----- drivers/staging/lustre/lnet/lnet/lib-msg.c | 16 +-- drivers/staging/lustre/lnet/lnet/lib-ptl.c | 7 + drivers/staging/lustre/lnet/lnet/net_fault.c | 24 ++-- drivers/staging/lustre/lnet/lnet/nidstrings.c | 9 +- drivers/staging/lustre/lnet/lnet/peer.c | 24 ++-- drivers/staging/lustre/lnet/lnet/router.c | 13 +- drivers/staging/lustre/lnet/selftest/conrpc.c | 5 - drivers/staging/lustre/lnet/selftest/console.c | 39 +++---- drivers/staging/lustre/lnet/selftest/framework.c | 44 ++++---- drivers/staging/lustre/lnet/selftest/rpc.c | 27 ++--- drivers/staging/lustre/lnet/selftest/timer.c | 4 - drivers/staging/lustre/lustre/include/cl_object.h | 19 +-- .../staging/lustre/lustre/include/lustre_debug.h | 52 --------- .../staging/lustre/lustre/include/lustre_export.h | 2 drivers/staging/lustre/lustre/include/obd.h | 1 drivers/staging/lustre/lustre/include/obd_class.h | 3 - drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 10 +- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 4 - drivers/staging/lustre/lustre/llite/dir.c | 17 +-- .../staging/lustre/lustre/llite/llite_internal.h | 2 drivers/staging/lustre/lustre/llite/llite_lib.c | 8 + drivers/staging/lustre/lustre/llite/lproc_llite.c | 2 drivers/staging/lustre/lustre/llite/statahead.c | 23 ++-- drivers/staging/lustre/lustre/llite/vvp_page.c | 9 +- drivers/staging/lustre/lustre/lov/lov_io.c | 9 +- drivers/staging/lustre/lustre/lov/lov_obd.c | 15 +-- drivers/staging/lustre/lustre/lov/lov_page.c | 3 - drivers/staging/lustre/lustre/obdclass/Makefile | 2 drivers/staging/lustre/lustre/obdclass/cl_io.c | 91 ++++------------ drivers/staging/lustre/lustre/obdclass/cl_lock.c | 5 - drivers/staging/lustre/lustre/obdclass/cl_object.c | 110 ++++++++------------ drivers/staging/lustre/lustre/obdclass/cl_page.c | 77 ++++---------- drivers/staging/lustre/lustre/obdclass/class_obd.c | 32 ++---- drivers/staging/lustre/lustre/obdclass/debug.c | 96 ----------------- drivers/staging/lustre/lustre/obdclass/genops.c | 90 ++++++++++------ drivers/staging/lustre/lustre/obdclass/lu_object.c | 7 + .../staging/lustre/lustre/obdclass/lustre_peer.c | 5 - drivers/staging/lustre/lustre/obdecho/Makefile | 2 drivers/staging/lustre/lustre/obdecho/debug.c | 96 +++++++++++++++++ .../staging/lustre/lustre/obdecho/echo_client.c | 3 - .../staging/lustre/lustre/obdecho/echo_internal.h | 4 + drivers/staging/lustre/lustre/osc/osc_cache.c | 17 ++- drivers/staging/lustre/lustre/osc/osc_lock.c | 7 + drivers/staging/lustre/lustre/osc/osc_page.c | 21 ++-- drivers/staging/lustre/lustre/osc/osc_request.c | 11 +- drivers/staging/lustre/lustre/ptlrpc/client.c | 14 +-- drivers/staging/lustre/lustre/ptlrpc/layout.c | 1 drivers/staging/lustre/lustre/ptlrpc/nrs_fifo.c | 6 + .../staging/lustre/lustre/ptlrpc/pack_generic.c | 4 - drivers/staging/lustre/lustre/ptlrpc/sec_gc.c | 6 + drivers/staging/lustre/lustre/ptlrpc/service.c | 68 ++++++------ 60 files changed, 689 insertions(+), 847 deletions(-) delete mode 100644 drivers/staging/lustre/lustre/include/lustre_debug.h delete mode 100644 drivers/staging/lustre/lustre/obdclass/debug.c create mode 100644 drivers/staging/lustre/lustre/obdecho/debug.c -- Signature