From patchwork Mon Feb 18 03:22:29 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Amerigo Wang X-Patchwork-Id: 221138 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 8F3292C008C for ; Mon, 18 Feb 2013 14:22:55 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755847Ab3BRDWw (ORCPT ); Sun, 17 Feb 2013 22:22:52 -0500 Received: from mx1.redhat.com ([209.132.183.28]:65395 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755212Ab3BRDWv (ORCPT ); Sun, 17 Feb 2013 22:22:51 -0500 Received: from int-mx12.intmail.prod.int.phx2.redhat.com (int-mx12.intmail.prod.int.phx2.redhat.com [10.5.11.25]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r1I3Mjox003707 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Sun, 17 Feb 2013 22:22:46 -0500 Received: from cr0.redhat.com (vpn1-115-224.nay.redhat.com [10.66.115.224]) by int-mx12.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id r1I3Mc7e019523; Sun, 17 Feb 2013 22:22:40 -0500 From: Cong Wang To: netdev@vger.kernel.org Cc: Eric Dumazet , Jiri Pirko , Vlad Yasevich , Stephen Hemminger , "David S. Miller" , Cong Wang Subject: [Patch net-next v5] netpoll: fix some sparse warnings in various places Date: Mon, 18 Feb 2013 11:22:29 +0800 Message-Id: <1361157749-9367-1-git-send-email-amwang@redhat.com> X-Scanned-By: MIMEDefang 2.68 on 10.5.11.25 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Cong Wang This fixes sparse warnings like the one below: drivers/net/team/team.c:953:25: warning: incorrect type in return expression (different address spaces) drivers/net/team/team.c:953:25: expected struct netpoll_info * drivers/net/team/team.c:953:25: got struct netpoll_info [noderef] *npinfo and make these functions return bool as suggested by Jiri. Cc: Eric Dumazet Cc: Jiri Pirko Cc: Vlad Yasevich Cc: Stephen Hemminger Cc: David S. Miller Signed-off-by: Cong Wang --- -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c index 94c1534..2263bf2 100644 --- a/drivers/net/bonding/bond_main.c +++ b/drivers/net/bonding/bond_main.c @@ -1300,9 +1300,9 @@ static int bond_netpoll_setup(struct net_device *dev, struct netpoll_info *ni, g return err; } -static struct netpoll_info *bond_netpoll_info(struct bonding *bond) +static bool bond_netpoll_info(struct bonding *bond) { - return bond->dev->npinfo; + return rcu_access_pointer(bond->dev->npinfo) != NULL; } #else @@ -1862,8 +1862,8 @@ int bond_enslave(struct net_device *bond_dev, struct net_device *slave_dev) bond_set_carrier(bond); #ifdef CONFIG_NET_POLL_CONTROLLER - slave_dev->npinfo = bond_netpoll_info(bond); - if (slave_dev->npinfo) { + rcu_assign_pointer(slave_dev->npinfo, bond->dev->npinfo); + if (bond_netpoll_info(bond)) { if (slave_enable_netpoll(new_slave)) { read_unlock(&bond->lock); pr_info("Error, %s: master_dev is using netpoll, " diff --git a/drivers/net/team/team.c b/drivers/net/team/team.c index 05c5efe..26abee3 100644 --- a/drivers/net/team/team.c +++ b/drivers/net/team/team.c @@ -948,9 +948,9 @@ static void team_port_disable_netpoll(struct team_port *port) kfree(np); } -static struct netpoll_info *team_netpoll_info(struct team *team) +static bool team_netpoll_info(struct team *team) { - return team->dev->npinfo; + return rcu_access_pointer(team->dev->npinfo) != NULL; } #else @@ -962,9 +962,9 @@ static int team_port_enable_netpoll(struct team *team, struct team_port *port, static void team_port_disable_netpoll(struct team_port *port) { } -static struct netpoll_info *team_netpoll_info(struct team *team) +static bool team_netpoll_info(struct team *team) { - return NULL; + return false; } #endif diff --git a/net/bridge/br_private.h b/net/bridge/br_private.h index 6d314c4..f69cdda 100644 --- a/net/bridge/br_private.h +++ b/net/bridge/br_private.h @@ -328,9 +328,9 @@ extern void br_dev_delete(struct net_device *dev, struct list_head *list); extern netdev_tx_t br_dev_xmit(struct sk_buff *skb, struct net_device *dev); #ifdef CONFIG_NET_POLL_CONTROLLER -static inline struct netpoll_info *br_netpoll_info(struct net_bridge *br) +static inline bool br_netpoll_info(struct net_bridge *br) { - return br->dev->npinfo; + return rcu_access_pointer(br->dev->npinfo) != NULL; } static inline void br_netpoll_send_skb(const struct net_bridge_port *p, @@ -345,9 +345,9 @@ static inline void br_netpoll_send_skb(const struct net_bridge_port *p, extern int br_netpoll_enable(struct net_bridge_port *p, gfp_t gfp); extern void br_netpoll_disable(struct net_bridge_port *p); #else -static inline struct netpoll_info *br_netpoll_info(struct net_bridge *br) +static inline bool br_netpoll_info(struct net_bridge *br) { - return NULL; + return false; } static inline void br_netpoll_send_skb(const struct net_bridge_port *p,