All of lore.kernel.org
 help / color / mirror / Atom feed
From: James Simmons <jsimmons@infradead.org>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	devel@driverdev.osuosl.org,
	Andreas Dilger <andreas.dilger@intel.com>,
	Oleg Drokin <oleg.drokin@intel.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Lustre Development List <lustre-devel@lists.lustre.org>,
	James Simmons <jsimmons@infradead.org>,
	James Simmons <uja.ornl@yahoo.com>
Subject: [PATCH 08/34] staging: lustre: lnet: change lstcon_rpc_ent_t to proper structure
Date: Mon, 16 Jan 2017 16:30:11 -0500	[thread overview]
Message-ID: <1484602237-30867-9-git-send-email-jsimmons@infradead.org> (raw)
In-Reply-To: <1484602237-30867-1-git-send-email-jsimmons@infradead.org>

Change lstcon_rpc_ent_t from typedef to proper structure.

Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142
Reviewed-on: https://review.whamcloud.com/24188
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
Signed-off-by: James Simmons <jsimmons@infradead.org>
---
 drivers/staging/lustre/include/linux/lnet/lnetst.h | 4 ++--
 drivers/staging/lustre/lnet/selftest/conrpc.c      | 4 ++--
 drivers/staging/lustre/lnet/selftest/conrpc.h      | 2 +-
 drivers/staging/lustre/lnet/selftest/console.c     | 6 +++---
 4 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/drivers/staging/lustre/include/linux/lnet/lnetst.h b/drivers/staging/lustre/include/linux/lnet/lnetst.h
index adc03fe..f5c68c1 100644
--- a/drivers/staging/lustre/include/linux/lnet/lnetst.h
+++ b/drivers/staging/lustre/include/linux/lnet/lnetst.h
@@ -124,7 +124,7 @@ struct lstcon_test_batch_ent {
 };				/*** test/batch verbose information entry,
 				 *** for list_batch command */
 
-typedef struct {
+struct lstcon_rpc_ent {
 	struct list_head	rpe_link;	/* link chain */
 	lnet_process_id_t	rpe_peer;	/* peer's id */
 	struct timeval		rpe_stamp;	/* time stamp of RPC */
@@ -135,7 +135,7 @@ struct lstcon_test_batch_ent {
 	int			rpe_fwk_errno;	/* framework errno */
 	int			rpe_priv[4];	/* private data */
 	char			rpe_payload[0];	/* private reply payload */
-} lstcon_rpc_ent_t;
+};
 
 typedef struct {
 	int	trs_rpc_stat[4];	/* RPCs stat (0: total
diff --git a/drivers/staging/lustre/lnet/selftest/conrpc.c b/drivers/staging/lustre/lnet/selftest/conrpc.c
index 84ae010..bc924f9 100644
--- a/drivers/staging/lustre/lnet/selftest/conrpc.c
+++ b/drivers/staging/lustre/lnet/selftest/conrpc.c
@@ -469,7 +469,7 @@ void lstcon_rpc_stat_reply(struct lstcon_rpc_trans *, struct srpc_msg *,
 {
 	struct list_head tmp;
 	struct list_head __user *next;
-	lstcon_rpc_ent_t *ent;
+	struct lstcon_rpc_ent *ent;
 	struct srpc_generic_reply *rep;
 	struct lstcon_rpc *crpc;
 	struct srpc_msg *msg;
@@ -492,7 +492,7 @@ void lstcon_rpc_stat_reply(struct lstcon_rpc_trans *, struct srpc_msg *,
 
 		next = tmp.next;
 
-		ent = list_entry(next, lstcon_rpc_ent_t, rpe_link);
+		ent = list_entry(next, struct lstcon_rpc_ent, rpe_link);
 
 		LASSERT(crpc->crp_stamp);
 
diff --git a/drivers/staging/lustre/lnet/selftest/conrpc.h b/drivers/staging/lustre/lnet/selftest/conrpc.h
index e629e87..ad0a49e 100644
--- a/drivers/staging/lustre/lnet/selftest/conrpc.h
+++ b/drivers/staging/lustre/lnet/selftest/conrpc.h
@@ -103,7 +103,7 @@ struct lstcon_rpc_trans {
 
 typedef int (*lstcon_rpc_cond_func_t)(int, struct lstcon_node *, void *);
 typedef int (*lstcon_rpc_readent_func_t)(int, struct srpc_msg *,
-					 lstcon_rpc_ent_t __user *);
+					 struct lstcon_rpc_ent __user *);
 
 int  lstcon_sesrpc_prep(struct lstcon_node *nd, int transop,
 			unsigned int version, struct lstcon_rpc **crpc);
diff --git a/drivers/staging/lustre/lnet/selftest/console.c b/drivers/staging/lustre/lnet/selftest/console.c
index bf4111a..b01023c 100644
--- a/drivers/staging/lustre/lnet/selftest/console.c
+++ b/drivers/staging/lustre/lnet/selftest/console.c
@@ -368,7 +368,7 @@ static void lstcon_group_ndlink_release(struct lstcon_group *,
 
 static int
 lstcon_sesrpc_readent(int transop, struct srpc_msg *msg,
-		      lstcon_rpc_ent_t __user *ent_up)
+		      struct lstcon_rpc_ent __user *ent_up)
 {
 	struct srpc_debug_reply *rep;
 
@@ -1385,7 +1385,7 @@ static void lstcon_group_ndlink_release(struct lstcon_group *,
 
 static int
 lstcon_tsbrpc_readent(int transop, struct srpc_msg *msg,
-		      lstcon_rpc_ent_t __user *ent_up)
+		      struct lstcon_rpc_ent __user *ent_up)
 {
 	struct srpc_batch_reply *rep = &msg->msg_body.bat_reply;
 
@@ -1464,7 +1464,7 @@ static void lstcon_group_ndlink_release(struct lstcon_group *,
 
 static int
 lstcon_statrpc_readent(int transop, struct srpc_msg *msg,
-		       lstcon_rpc_ent_t __user *ent_up)
+		       struct lstcon_rpc_ent __user *ent_up)
 {
 	struct srpc_stat_reply *rep = &msg->msg_body.stat_reply;
 	sfw_counters_t __user *sfwk_stat;
-- 
1.8.3.1

WARNING: multiple messages have this Message-ID (diff)
From: James Simmons <jsimmons@infradead.org>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	devel@driverdev.osuosl.org,
	Andreas Dilger <andreas.dilger@intel.com>,
	Oleg Drokin <oleg.drokin@intel.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Lustre Development List <lustre-devel@lists.lustre.org>,
	James Simmons <jsimmons@infradead.org>,
	James Simmons <uja.ornl@yahoo.com>
Subject: [lustre-devel] [PATCH 08/34] staging: lustre: lnet: change lstcon_rpc_ent_t to proper structure
Date: Mon, 16 Jan 2017 16:30:11 -0500	[thread overview]
Message-ID: <1484602237-30867-9-git-send-email-jsimmons@infradead.org> (raw)
In-Reply-To: <1484602237-30867-1-git-send-email-jsimmons@infradead.org>

Change lstcon_rpc_ent_t from typedef to proper structure.

Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142
Reviewed-on: https://review.whamcloud.com/24188
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
Signed-off-by: James Simmons <jsimmons@infradead.org>
---
 drivers/staging/lustre/include/linux/lnet/lnetst.h | 4 ++--
 drivers/staging/lustre/lnet/selftest/conrpc.c      | 4 ++--
 drivers/staging/lustre/lnet/selftest/conrpc.h      | 2 +-
 drivers/staging/lustre/lnet/selftest/console.c     | 6 +++---
 4 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/drivers/staging/lustre/include/linux/lnet/lnetst.h b/drivers/staging/lustre/include/linux/lnet/lnetst.h
index adc03fe..f5c68c1 100644
--- a/drivers/staging/lustre/include/linux/lnet/lnetst.h
+++ b/drivers/staging/lustre/include/linux/lnet/lnetst.h
@@ -124,7 +124,7 @@ struct lstcon_test_batch_ent {
 };				/*** test/batch verbose information entry,
 				 *** for list_batch command */
 
-typedef struct {
+struct lstcon_rpc_ent {
 	struct list_head	rpe_link;	/* link chain */
 	lnet_process_id_t	rpe_peer;	/* peer's id */
 	struct timeval		rpe_stamp;	/* time stamp of RPC */
@@ -135,7 +135,7 @@ struct lstcon_test_batch_ent {
 	int			rpe_fwk_errno;	/* framework errno */
 	int			rpe_priv[4];	/* private data */
 	char			rpe_payload[0];	/* private reply payload */
-} lstcon_rpc_ent_t;
+};
 
 typedef struct {
 	int	trs_rpc_stat[4];	/* RPCs stat (0: total
diff --git a/drivers/staging/lustre/lnet/selftest/conrpc.c b/drivers/staging/lustre/lnet/selftest/conrpc.c
index 84ae010..bc924f9 100644
--- a/drivers/staging/lustre/lnet/selftest/conrpc.c
+++ b/drivers/staging/lustre/lnet/selftest/conrpc.c
@@ -469,7 +469,7 @@ void lstcon_rpc_stat_reply(struct lstcon_rpc_trans *, struct srpc_msg *,
 {
 	struct list_head tmp;
 	struct list_head __user *next;
-	lstcon_rpc_ent_t *ent;
+	struct lstcon_rpc_ent *ent;
 	struct srpc_generic_reply *rep;
 	struct lstcon_rpc *crpc;
 	struct srpc_msg *msg;
@@ -492,7 +492,7 @@ void lstcon_rpc_stat_reply(struct lstcon_rpc_trans *, struct srpc_msg *,
 
 		next = tmp.next;
 
-		ent = list_entry(next, lstcon_rpc_ent_t, rpe_link);
+		ent = list_entry(next, struct lstcon_rpc_ent, rpe_link);
 
 		LASSERT(crpc->crp_stamp);
 
diff --git a/drivers/staging/lustre/lnet/selftest/conrpc.h b/drivers/staging/lustre/lnet/selftest/conrpc.h
index e629e87..ad0a49e 100644
--- a/drivers/staging/lustre/lnet/selftest/conrpc.h
+++ b/drivers/staging/lustre/lnet/selftest/conrpc.h
@@ -103,7 +103,7 @@ struct lstcon_rpc_trans {
 
 typedef int (*lstcon_rpc_cond_func_t)(int, struct lstcon_node *, void *);
 typedef int (*lstcon_rpc_readent_func_t)(int, struct srpc_msg *,
-					 lstcon_rpc_ent_t __user *);
+					 struct lstcon_rpc_ent __user *);
 
 int  lstcon_sesrpc_prep(struct lstcon_node *nd, int transop,
 			unsigned int version, struct lstcon_rpc **crpc);
diff --git a/drivers/staging/lustre/lnet/selftest/console.c b/drivers/staging/lustre/lnet/selftest/console.c
index bf4111a..b01023c 100644
--- a/drivers/staging/lustre/lnet/selftest/console.c
+++ b/drivers/staging/lustre/lnet/selftest/console.c
@@ -368,7 +368,7 @@ static void lstcon_group_ndlink_release(struct lstcon_group *,
 
 static int
 lstcon_sesrpc_readent(int transop, struct srpc_msg *msg,
-		      lstcon_rpc_ent_t __user *ent_up)
+		      struct lstcon_rpc_ent __user *ent_up)
 {
 	struct srpc_debug_reply *rep;
 
@@ -1385,7 +1385,7 @@ static void lstcon_group_ndlink_release(struct lstcon_group *,
 
 static int
 lstcon_tsbrpc_readent(int transop, struct srpc_msg *msg,
-		      lstcon_rpc_ent_t __user *ent_up)
+		      struct lstcon_rpc_ent __user *ent_up)
 {
 	struct srpc_batch_reply *rep = &msg->msg_body.bat_reply;
 
@@ -1464,7 +1464,7 @@ static void lstcon_group_ndlink_release(struct lstcon_group *,
 
 static int
 lstcon_statrpc_readent(int transop, struct srpc_msg *msg,
-		       lstcon_rpc_ent_t __user *ent_up)
+		       struct lstcon_rpc_ent __user *ent_up)
 {
 	struct srpc_stat_reply *rep = &msg->msg_body.stat_reply;
 	sfw_counters_t __user *sfwk_stat;
-- 
1.8.3.1

  parent reply	other threads:[~2017-01-16 21:31 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-16 21:30 [PATCH 00/34] staging: lustre: lnet: change lnet selftest UAPI typedefs to proper structure James Simmons
2017-01-16 21:30 ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 01/34] staging: lustre: lnet: change lst_nid_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 02/34] staging: lustre: lnet: change lst_bid_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 03/34] staging: lustre: lnet: change lstcon_node_ent_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 04/34] staging: lustre: lnet: change lstcon_ndlist_ent_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 05/34] staging: lustre: lnet: change lstcon_test_ent_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 06/34] staging: lustre: lnet: change lstcon_batch_ent_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 07/34] staging: lustre: lnet: change lstcon_test_batch_ent_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` James Simmons [this message]
2017-01-16 21:30   ` [lustre-devel] [PATCH 08/34] staging: lustre: lnet: change lstcon_rpc_ent_t " James Simmons
2017-01-16 21:30 ` [PATCH 09/34] staging: lustre: lnet: change lstcon_trans_stat_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 10/34] staging: lustre: lnet: change lstio_session_new_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 11/34] staging: lustre: lnet: change lstio_session_info_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 12/34] staging: lustre: lnet: change lstio_session_end_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 13/34] staging: lustre: lnet: change lstio_debug_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 14/34] staging: lustre: lnet: change lstio_group_add_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 15/34] staging: lustre: lnet: change lstio_group_del_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 16/34] staging: lustre: lnet: change lstio_group_update_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 17/34] staging: lustre: lnet: change lstio_group_nodes_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 18/34] staging: lustre: lnet: change lstio_group_list_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 19/34] staging: lustre: lnet: change lstio_group_info_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 20/34] staging: lustre: lnet: change lstio_batch_add_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 21/34] staging: lustre: lnet: change lstio_batch_del_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 22/34] staging: lustre: lnet: change lstio_batch_run_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 23/34] staging: lustre: lnet: change lstio_batch_stop_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 24/34] staging: lustre: lnet: change lstio_batch_query_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 25/34] staging: lustre: lnet: change lstio_batch_list_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 26/34] staging: lustre: lnet: change lstio_batch_info_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 27/34] staging: lustre: lnet: change lstio_stat_args_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 28/34] staging: lustre: lnet: change lst_test_type_t to proper enum James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 29/34] staging: lustre: lnet: change lstio_test_args_t to proper structure James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 30/34] staging: lustre: lnet: change lst_brw_[type|flags]_t to proper enum James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 31/34] staging: lustre: lnet: change lst_test_bulk_param_t to proper structure James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 32/34] staging: lustre: lnet: change lst_test_ping_param_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 33/34] staging: lustre: lnet: change srpc_counters_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons
2017-01-16 21:30 ` [PATCH 34/34] staging: lustre: lnet: change sfw_counter_t " James Simmons
2017-01-16 21:30   ` [lustre-devel] " James Simmons

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=1484602237-30867-9-git-send-email-jsimmons@infradead.org \
    --to=jsimmons@infradead.org \
    --cc=andreas.dilger@intel.com \
    --cc=devel@driverdev.osuosl.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lustre-devel@lists.lustre.org \
    --cc=oleg.drokin@intel.com \
    --cc=uja.ornl@yahoo.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.