[PATCH net-next] networking: use skb_put_zero()

[PATCH net-next] networking: use skb_put_zero()

[Johannes Berg]

From: Johannes Berg <johannes.berg@intel.com>

Use the recently introduced helper to replace the pattern of
skb_put() && memset(), this transformation was done with the
following spatch:

@@
identifier p;
expression len;
expression skb;
@@
-p = skb_put(skb, len);
-memset(p, 0, len);
+p = skb_put_zero(skb, len);

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
---
This obviously requires skb_put_zero() which is coming in through
the mac80211-next merge I just sent you a pull request for.
---
 drivers/net/ethernet/apple/bmac.c            | 3 +--
 drivers/net/ethernet/broadcom/bcm63xx_enet.c | 3 +--
 drivers/net/ethernet/qualcomm/qca_spi.c      | 3 +--
 drivers/net/wireless/marvell/mwifiex/tdls.c  | 4 ++--
 drivers/nfc/pn533/pn533.c                    | 3 +--
 lib/nlattr.c                                 | 3 +--
 net/sctp/sm_make_chunk.c                     | 3 +--
 7 files changed, 8 insertions(+), 14 deletions(-)

diff --git a/drivers/net/ethernet/apple/bmac.c b/drivers/net/ethernet/apple/bmac.c
index 2b2d87089987..eac740c476ce 100644
--- a/drivers/net/ethernet/apple/bmac.c
+++ b/drivers/net/ethernet/apple/bmac.c
@@ -1218,8 +1218,7 @@ static void bmac_reset_and_enable(struct net_device *dev)
 	 */
 	skb = netdev_alloc_skb(dev, ETHERMINPACKET);
 	if (skb != NULL) {
-		data = skb_put(skb, ETHERMINPACKET);
-		memset(data, 0, ETHERMINPACKET);
+		data = skb_put_zero(skb, ETHERMINPACKET);
 		memcpy(data, dev->dev_addr, ETH_ALEN);
 		memcpy(data + ETH_ALEN, dev->dev_addr, ETH_ALEN);
 		bmac_transmit_packet(skb, dev);
diff --git a/drivers/net/ethernet/broadcom/bcm63xx_enet.c b/drivers/net/ethernet/broadcom/bcm63xx_enet.c
index 50d88d3e03b6..18edc2622135 100644
--- a/drivers/net/ethernet/broadcom/bcm63xx_enet.c
+++ b/drivers/net/ethernet/broadcom/bcm63xx_enet.c
@@ -609,8 +609,7 @@ static int bcm_enet_start_xmit(struct sk_buff *skb, struct net_device *dev)
 			dev_kfree_skb(skb);
 			skb = nskb;
 		}
-		data = skb_put(skb, needed);
-		memset(data, 0, needed);
+		data = skb_put_zero(skb, needed);
 	}
 
 	/* point to the next available desc */
diff --git a/drivers/net/ethernet/qualcomm/qca_spi.c b/drivers/net/ethernet/qualcomm/qca_spi.c
index de78f60309a0..9c236298fe21 100644
--- a/drivers/net/ethernet/qualcomm/qca_spi.c
+++ b/drivers/net/ethernet/qualcomm/qca_spi.c
@@ -719,8 +719,7 @@ qcaspi_netdev_xmit(struct sk_buff *skb, struct net_device *dev)
 	qcafrm_create_header(ptmp, frame_len);
 
 	if (pad_len) {
-		ptmp = skb_put(skb, pad_len);
-		memset(ptmp, 0, pad_len);
+		ptmp = skb_put_zero(skb, pad_len);
 	}
 
 	ptmp = skb_put(skb, QCAFRM_FOOTER_LEN);
diff --git a/drivers/net/wireless/marvell/mwifiex/tdls.c b/drivers/net/wireless/marvell/mwifiex/tdls.c
index d76ce8797de1..b7d124dbef0c 100644
--- a/drivers/net/wireless/marvell/mwifiex/tdls.c
+++ b/drivers/net/wireless/marvell/mwifiex/tdls.c
@@ -853,8 +853,8 @@ int mwifiex_send_tdls_action_frame(struct mwifiex_private *priv, const u8 *peer,
 
 	pkt_type = PKT_TYPE_MGMT;
 	tx_control = 0;
-	pos = skb_put(skb, MWIFIEX_MGMT_FRAME_HEADER_SIZE + sizeof(pkt_len));
-	memset(pos, 0, MWIFIEX_MGMT_FRAME_HEADER_SIZE + sizeof(pkt_len));
+	pos = skb_put_zero(skb,
+			   MWIFIEX_MGMT_FRAME_HEADER_SIZE + sizeof(pkt_len));
 	memcpy(pos, &pkt_type, sizeof(pkt_type));
 	memcpy(pos + sizeof(pkt_type), &tx_control, sizeof(tx_control));
 
diff --git a/drivers/nfc/pn533/pn533.c b/drivers/nfc/pn533/pn533.c
index 65bbaa5fcdda..70c304504a29 100644
--- a/drivers/nfc/pn533/pn533.c
+++ b/drivers/nfc/pn533/pn533.c
@@ -1043,8 +1043,7 @@ static struct sk_buff *pn533_alloc_poll_tg_frame(struct pn533 *dev)
 	get_random_bytes(felica + 2, 6);
 
 	/* NFCID3 */
-	nfcid3 = skb_put(skb, 10);
-	memset(nfcid3, 0, 10);
+	nfcid3 = skb_put_zero(skb, 10);
 	memcpy(nfcid3, felica, 8);
 
 	/* General bytes */
diff --git a/lib/nlattr.c b/lib/nlattr.c
index a7e0b16078df..d09d9746fc5d 100644
--- a/lib/nlattr.c
+++ b/lib/nlattr.c
@@ -400,8 +400,7 @@ void *__nla_reserve_nohdr(struct sk_buff *skb, int attrlen)
 {
 	void *start;
 
-	start = skb_put(skb, NLA_ALIGN(attrlen));
-	memset(start, 0, NLA_ALIGN(attrlen));
+	start = skb_put_zero(skb, NLA_ALIGN(attrlen));
 
 	return start;
 }
diff --git a/net/sctp/sm_make_chunk.c b/net/sctp/sm_make_chunk.c
index bd439edf2d8a..ea2601501654 100644
--- a/net/sctp/sm_make_chunk.c
+++ b/net/sctp/sm_make_chunk.c
@@ -1296,8 +1296,7 @@ struct sctp_chunk *sctp_make_auth(const struct sctp_association *asoc)
 	retval->subh.auth_hdr = sctp_addto_chunk(retval, sizeof(sctp_authhdr_t),
 						&auth_hdr);
 
-	hmac = skb_put(retval->skb, hmac_desc->hmac_len);
-	memset(hmac, 0, hmac_desc->hmac_len);
+	hmac = skb_put_zero(retval->skb, hmac_desc->hmac_len);
 
 	/* Adjust the chunk header to include the empty MAC */
 	retval->chunk_hdr->length =
-- 
2.11.0

Re: [PATCH net-next] networking: use skb_put_zero()

[David Miller]

From: Johannes Berg <johannes@sipsolutions.net>
Date: Tue, 13 Jun 2017 14:28:18 +0200

> From: Johannes Berg <johannes.berg@intel.com>
> 
> Use the recently introduced helper to replace the pattern of
> skb_put() && memset(), this transformation was done with the
> following spatch:
> 
> @@
> identifier p;
> expression len;
> expression skb;
> @@
> -p = skb_put(skb, len);
> -memset(p, 0, len);
> +p = skb_put_zero(skb, len);
> 
> Signed-off-by: Johannes Berg <johannes.berg@intel.com>

Applied, thanks!

Re: [PATCH net-next] networking: use skb_put_zero()

[kbuild test robot]

[-- Attachment #1: Type: text/plain, Size: 1911 bytes --]

Hi Johannes,

[auto build test ERROR on net-next/master]

url:    https://github.com/0day-ci/linux/commits/Johannes-Berg/networking-use-skb_put_zero/20170614-035553
config: x86_64-randconfig-x019-201724 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All error/warnings (new ones prefixed by >>):

   drivers/nfc/pn533/pn533.c: In function 'pn533_alloc_poll_tg_frame':
>> drivers/nfc/pn533/pn533.c:1046:11: error: implicit declaration of function 'skb_put_zero' [-Werror=implicit-function-declaration]
     nfcid3 = skb_put_zero(skb, 10);
              ^~~~~~~~~~~~
>> drivers/nfc/pn533/pn533.c:1046:9: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
     nfcid3 = skb_put_zero(skb, 10);
            ^
   cc1: some warnings being treated as errors
--
   net/sctp/sm_make_chunk.c: In function 'sctp_make_auth':
>> net/sctp/sm_make_chunk.c:1299:9: error: implicit declaration of function 'skb_put_zero' [-Werror=implicit-function-declaration]
     hmac = skb_put_zero(retval->skb, hmac_desc->hmac_len);
            ^~~~~~~~~~~~
>> net/sctp/sm_make_chunk.c:1299:7: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
     hmac = skb_put_zero(retval->skb, hmac_desc->hmac_len);
          ^
   cc1: some warnings being treated as errors

vim +/skb_put_zero +1046 drivers/nfc/pn533/pn533.c

  1040		/* Felica params */
  1041		felica = skb_put(skb, 18);
  1042		memcpy(felica, felica_params, 18);
  1043		get_random_bytes(felica + 2, 6);
  1044	
  1045		/* NFCID3 */
> 1046		nfcid3 = skb_put_zero(skb, 10);
  1047		memcpy(nfcid3, felica, 8);
  1048	
  1049		/* General bytes */

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 33788 bytes --]

Re: [PATCH net-next] networking: use skb_put_zero()

[kbuild test robot]

[-- Attachment #1: Type: text/plain, Size: 1324 bytes --]

Hi Johannes,

[auto build test ERROR on net-next/master]

url:    https://github.com/0day-ci/linux/commits/Johannes-Berg/networking-use-skb_put_zero/20170614-035553
config: x86_64-randconfig-x010-201724 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All error/warnings (new ones prefixed by >>):

   lib/nlattr.c: In function '__nla_reserve_nohdr':
>> lib/nlattr.c:403:10: error: implicit declaration of function 'skb_put_zero' [-Werror=implicit-function-declaration]
     start = skb_put_zero(skb, NLA_ALIGN(attrlen));
             ^~~~~~~~~~~~
>> lib/nlattr.c:403:8: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
     start = skb_put_zero(skb, NLA_ALIGN(attrlen));
           ^
   cc1: some warnings being treated as errors

vim +/skb_put_zero +403 lib/nlattr.c

   397	 * tailroom for the payload.
   398	 */
   399	void *__nla_reserve_nohdr(struct sk_buff *skb, int attrlen)
   400	{
   401		void *start;
   402	
 > 403		start = skb_put_zero(skb, NLA_ALIGN(attrlen));
   404	
   405		return start;
   406	}

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 22864 bytes --]

Re: [PATCH net-next] networking: use skb_put_zero()

[David Miller]

These are bogus reports I think.

Johannes's changes depends upon the wireless GIT tree I pulled in
right before I applied his patch.

Re: [kbuild-all] [PATCH net-next] networking: use skb_put_zero()

[Ye Xiaolong]

Hi, David

On 06/13, David Miller wrote:
>
>These are bogus reports I think.
>
>Johannes's changes depends upon the wireless GIT tree I pulled in
>right before I applied his patch.

Did you applied the patch on top of master branch?

Thanks,
Xiaolong

>_______________________________________________
>kbuild-all mailing list
>kbuild-all@lists.01.org
>https://lists.01.org/mailman/listinfo/kbuild-all

Re: [kbuild-all] [PATCH net-next] networking: use skb_put_zero()

[Johannes Berg]

On Fri, 2017-06-16 at 16:30 +0800, Ye Xiaolong wrote:
> Hi, David
> 
> On 06/13, David Miller wrote:
> > 
> > These are bogus reports I think.
> > 
> > Johannes's changes depends upon the wireless GIT tree I pulled in
> > right before I applied his patch.
> 
> Did you applied the patch on top of master branch?

Yes, but after merging my git tree.

There was no way the bot could figure this out - it was a discussion
we'd had earlier, and when I sent the pull request I also sent out the
patch a few minutes later.

johannes