Re: [PATCH] tuntap: fix sparse warning

2012-12-17 Thread David Miller
From: Jason Wang 
Date: Tue, 18 Dec 2012 11:00:27 +0800

> Make tun_enable_queue() static to fix the sparse warning:
> 
> drivers/net/tun.c:399:19: sparse: symbol 'tun_enable_queue' was not declared. 
> Should it be static?
> 
> Reported-by: Fengguang Wu 
> Signed-off-by: Jason Wang 

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


[PATCH] tuntap: fix sparse warning

2012-12-17 Thread Jason Wang
Make tun_enable_queue() static to fix the sparse warning:

drivers/net/tun.c:399:19: sparse: symbol 'tun_enable_queue' was not declared. 
Should it be static?

Reported-by: Fengguang Wu 
Signed-off-by: Jason Wang 
---
 drivers/net/tun.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/net/tun.c b/drivers/net/tun.c
index 173acf5..504f7f1 100644
--- a/drivers/net/tun.c
+++ b/drivers/net/tun.c
@@ -396,7 +396,7 @@ static void tun_disable_queue(struct tun_struct *tun, 
struct tun_file *tfile)
++tun->numdisabled;
 }
 
-struct tun_struct *tun_enable_queue(struct tun_file *tfile)
+static struct tun_struct *tun_enable_queue(struct tun_file *tfile)
 {
struct tun_struct *tun = tfile->detached;
 
-- 
1.7.1

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


[PATCH] tuntap: fix sparse warning

2012-12-17 Thread Jason Wang
Make tun_enable_queue() static to fix the sparse warning:

drivers/net/tun.c:399:19: sparse: symbol 'tun_enable_queue' was not declared. 
Should it be static?

Reported-by: Fengguang Wu fengguang...@intel.com
Signed-off-by: Jason Wang jasow...@redhat.com
---
 drivers/net/tun.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/net/tun.c b/drivers/net/tun.c
index 173acf5..504f7f1 100644
--- a/drivers/net/tun.c
+++ b/drivers/net/tun.c
@@ -396,7 +396,7 @@ static void tun_disable_queue(struct tun_struct *tun, 
struct tun_file *tfile)
++tun-numdisabled;
 }
 
-struct tun_struct *tun_enable_queue(struct tun_file *tfile)
+static struct tun_struct *tun_enable_queue(struct tun_file *tfile)
 {
struct tun_struct *tun = tfile-detached;
 
-- 
1.7.1

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


Re: [PATCH] tuntap: fix sparse warning

2012-12-17 Thread David Miller
From: Jason Wang jasow...@redhat.com
Date: Tue, 18 Dec 2012 11:00:27 +0800

 Make tun_enable_queue() static to fix the sparse warning:
 
 drivers/net/tun.c:399:19: sparse: symbol 'tun_enable_queue' was not declared. 
 Should it be static?
 
 Reported-by: Fengguang Wu fengguang...@intel.com
 Signed-off-by: Jason Wang jasow...@redhat.com

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