Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1238642 > unrolled thread

[PATCH] staging/lustre: Make nrs_policy_get_info_locked() static

Started byRocco Folino <rocco@devzen.net>
First post2015-10-03 00:00 +0200
Last post2015-10-03 00:00 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] staging/lustre: Make nrs_policy_get_info_locked() static Rocco Folino <rocco@devzen.net> - 2015-10-03 00:00 +0200
    Re: [PATCH] staging/lustre: Make nrs_policy_get_info_locked() static Arnd Bergmann <arnd@arndb.de> - 2015-10-03 00:00 +0200

#1238642 — [PATCH] staging/lustre: Make nrs_policy_get_info_locked() static

FromRocco Folino <rocco@devzen.net>
Date2015-10-03 00:00 +0200
Subject[PATCH] staging/lustre: Make nrs_policy_get_info_locked() static
Message-ID<qfgwX-7vm-17@gated-at.bofh.it>
This patch fixes the warning generated by sparse: "symbol 'nrs_policy_get_info_locked' was not
declared. Should it be static?" by declaring the function static.

Signed-off-by: Rocco Folino <rocco@devzen.net>
---
 drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c b/drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c
index 3a212b4be9a1..6cf9b92c7c05 100644
--- a/drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c
+++ b/drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c
@@ -452,7 +452,7 @@ static const char *nrs_state2str(enum ptlrpc_nrs_pol_state state)
  * \param[in] policy The policy
  * \param[out] info  Holds returned status information
  */
-void nrs_policy_get_info_locked(struct ptlrpc_nrs_policy *policy,
+static void nrs_policy_get_info_locked(struct ptlrpc_nrs_policy *policy,
 				struct ptlrpc_nrs_pol_info *info)
 {
 	LASSERT(policy != NULL);
--
2.4.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1238644

FromArnd Bergmann <arnd@arndb.de>
Date2015-10-03 00:00 +0200
Message-ID<qfgwX-7vm-23@gated-at.bofh.it>
In reply to#1238642
On Friday 02 October 2015 23:54:26 Rocco Folino wrote:
> This patch fixes the warning generated by sparse: "symbol 'nrs_policy_get_info_locked' was not
> declared. Should it be static?" by declaring the function static.
> 
> Signed-off-by: Rocco Folino <rocco@devzen.net>
> 

Reviewed-by: Arnd Bergmann <arnd@arndb.de>

This probably triggered a sparse warning after the unused declaration was
removed in my "staging/lustre: remove lots of dead code".
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web