From patchwork Sun Aug 7 14:30:40 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alexander Aring X-Patchwork-Id: 9266395 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 1A67860754 for ; Sun, 7 Aug 2016 14:33:30 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 0C5F726C2F for ; Sun, 7 Aug 2016 14:33:30 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 0129027E5A; Sun, 7 Aug 2016 14:33:29 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 9F7DC26C2F for ; Sun, 7 Aug 2016 14:33:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751980AbcHGOd1 (ORCPT ); Sun, 7 Aug 2016 10:33:27 -0400 Received: from metis.ext.4.pengutronix.de ([92.198.50.35]:40802 "EHLO metis.ext.4.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751981AbcHGOdD (ORCPT ); Sun, 7 Aug 2016 10:33:03 -0400 Received: from gallifrey.ext.pengutronix.de ([2001:67c:670:201:5054:ff:fe8d:eefb] helo=omega.localdomain) by metis.ext.pengutronix.de with esmtp (Exim 4.80) (envelope-from ) id 1bWP8P-0001Um-5k; Sun, 07 Aug 2016 16:33:01 +0200 From: Alexander Aring To: linux-wpan@vger.kernel.org Cc: kernel@pengutronix.de, linux-bluetooth@vger.kernel.org, Alexander Aring Subject: [RFCv2 bluetooth-next 03/19] 6lowpan: remove LOWPAN_IPHC_MAX_HEADER_LEN Date: Sun, 7 Aug 2016 16:30:40 +0200 Message-Id: <20160807143056.3116-4-aar@pengutronix.de> X-Mailer: git-send-email 2.9.2 In-Reply-To: <20160807143056.3116-1-aar@pengutronix.de> References: <20160807143056.3116-1-aar@pengutronix.de> X-SA-Exim-Connect-IP: 2001:67c:670:201:5054:ff:fe8d:eefb X-SA-Exim-Mail-From: aar@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-wpan@vger.kernel.org Sender: linux-wpan-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wpan@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP My observations with btle 6lowpan was that such parameter doesn't exists. IPHC will not increase the packet size of IPv6 afterwards, but decrease it. I thought about that LOWPAN_IPHC are the 2 bytes headers, but we will always save two bytes because the payload_length of ipv6 will be elided. Then I thought the CID byte will increase the payload at one byte, this is correct but when we add the CID byte, we need to compress at worst-case 8 bytes. After that I thought the nhc id will be at least increase currently the payload to one byte. This is correct, but when we set the NHC bit, then we save one byte because ipv6 next header byte will be elided. That's why everything should be fine until we have NHC's which have two bytes length until or NHC compression use more bytes than raw next header format. Signed-off-by: Alexander Aring --- include/net/6lowpan.h | 7 ------- net/ieee802154/6lowpan/core.c | 6 ++---- 2 files changed, 2 insertions(+), 11 deletions(-) diff --git a/include/net/6lowpan.h b/include/net/6lowpan.h index 6d56be7..7f4b63c 100644 --- a/include/net/6lowpan.h +++ b/include/net/6lowpan.h @@ -68,15 +68,8 @@ * possible inline data. */ #define LOWPAN_NHC_MAX_HDR_LEN (sizeof(struct udphdr)) -/* Max IPHC Header len without IPv6 hdr specific inline data. - * Useful for getting the "extra" bytes we need at worst case compression. - * - * LOWPAN_IPHC + CID + LOWPAN_NHC_MAX_ID_LEN - */ -#define LOWPAN_IPHC_MAX_HEADER_LEN (2 + 1 + LOWPAN_NHC_MAX_ID_LEN) /* Maximum worst case IPHC header buffer size */ #define LOWPAN_IPHC_MAX_HC_BUF_LEN (sizeof(struct ipv6hdr) + \ - LOWPAN_IPHC_MAX_HEADER_LEN + \ LOWPAN_NHC_MAX_HDR_LEN) /* SCI/DCI is 4 bit width, so we have maximum 16 entries */ #define LOWPAN_IPHC_CTX_TABLE_SIZE (1 << 4) diff --git a/net/ieee802154/6lowpan/core.c b/net/ieee802154/6lowpan/core.c index 6afb8cd..f70edcc 100644 --- a/net/ieee802154/6lowpan/core.c +++ b/net/ieee802154/6lowpan/core.c @@ -152,11 +152,9 @@ static int lowpan_newlink(struct net *src_net, struct net_device *ldev, /* We need headroom for possible wpan_dev_hard_header call and tailroom * for encryption/fcs handling. The lowpan interface will replace * the IPv6 header with 6LoWPAN header. At worst case the 6LoWPAN - * header has LOWPAN_IPHC_MAX_HEADER_LEN more bytes than the IPv6 - * header. + * header has one byte more for possible raw IPv6 dispatch. */ - ldev->needed_headroom = LOWPAN_IPHC_MAX_HEADER_LEN + - wdev->needed_headroom; + ldev->needed_headroom = 1 + wdev->needed_headroom; ldev->needed_tailroom = wdev->needed_tailroom; ldev->neigh_priv_len = sizeof(struct lowpan_802154_neigh);