From patchwork Thu Dec 1 11:44:44 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Amir Vadai X-Patchwork-Id: 701474 X-Patchwork-Delegate: shemminger@vyatta.com 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 3tTwTc6rWYz9t0p for ; Thu, 1 Dec 2016 22:45:56 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755284AbcLALpy (ORCPT ); Thu, 1 Dec 2016 06:45:54 -0500 Received: from mail-wm0-f65.google.com ([74.125.82.65]:34039 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750738AbcLALpu (ORCPT ); Thu, 1 Dec 2016 06:45:50 -0500 Received: by mail-wm0-f65.google.com with SMTP id g23so33841066wme.1 for ; Thu, 01 Dec 2016 03:45:50 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=7Mu71+jhJAMaiA1IQmasbKvyfmuybASymnkCL2tesCA=; b=Vlecjg7KFYIcbSPKIG6Y2uDql8wIb9RMPXY0su3JxSaK6h/V0sDAFSpAhxC10k/VRi lAXwgdINyrg/l8ignVB1rvXaQpMNPWW4HQG9O5b1F4ktDlB/TyXuaKoHTLSvqujrn+U7 BxkI0gKpznFACJzYMMX0d89NYYOe2xswyocpJ9GvXr4MfXAieyErDoShFAgHSdx5OlEu XvUcZExEdMNJTHNXc0Nl4KLeLrI9jPovL8h1PWdeCdfjcbiJw0HEc+W12uPRu/+pOs4x vacpkepXA7JCoUqQOJjxwV0Dekh9t5DWFW6hL1iAkRJubp3b1RUWzbq0Jv9Q+cvFUAZZ dHQA== X-Gm-Message-State: AKaTC00fRzNT0vkwOdgRMeH4rKU9d014ikj4s+5N7uTCLXPEEwmOJ1h0JESF6Wzjmrer1g== X-Received: by 10.28.152.137 with SMTP id a131mr31240348wme.56.1480592749444; Thu, 01 Dec 2016 03:45:49 -0800 (PST) Received: from office.vadai.me ([192.116.94.210]) by smtp.gmail.com with ESMTPSA id v3sm47403919wjp.13.2016.12.01.03.45.47 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 01 Dec 2016 03:45:48 -0800 (PST) From: Amir Vadai To: Stephen Hemminger Cc: netdev@vger.kernel.org, "David S. Miller" , Jiri Benc , Or Gerlitz , Hadar Har-Zion , Roi Dayan , Amir Vadai Subject: [PATCH iproute2 V4 1/3] libnetlink: Introduce rta_getattr_be*() Date: Thu, 1 Dec 2016 13:44:44 +0200 Message-Id: <20161201114446.30333-2-amir@vadai.me> X-Mailer: git-send-email 2.10.2 In-Reply-To: <20161201114446.30333-1-amir@vadai.me> References: <20161201114446.30333-1-amir@vadai.me> Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Add the utility functions rta_getattr_be16() and rta_getattr_be32(), and change existing code to use it. Signed-off-by: Amir Vadai --- bridge/fdb.c | 4 ++-- include/libnetlink.h | 9 +++++++++ ip/iplink_geneve.c | 2 +- ip/iplink_vxlan.c | 2 +- 4 files changed, 13 insertions(+), 4 deletions(-) diff --git a/bridge/fdb.c b/bridge/fdb.c index 90f4b154c5dc..a91521776e99 100644 --- a/bridge/fdb.c +++ b/bridge/fdb.c @@ -168,10 +168,10 @@ int print_fdb(const struct sockaddr_nl *who, struct nlmsghdr *n, void *arg) if (tb[NDA_PORT]) { if (jw_global) jsonw_uint_field(jw_global, "port", - ntohs(rta_getattr_u16(tb[NDA_PORT]))); + rta_getattr_be16(tb[NDA_PORT])); else fprintf(fp, "port %d ", - ntohs(rta_getattr_u16(tb[NDA_PORT]))); + rta_getattr_be16(tb[NDA_PORT])); } if (tb[NDA_VNI]) { diff --git a/include/libnetlink.h b/include/libnetlink.h index 483509ca9635..751ebf186dd4 100644 --- a/include/libnetlink.h +++ b/include/libnetlink.h @@ -10,6 +10,7 @@ #include #include #include +#include struct rtnl_handle { int fd; @@ -140,10 +141,18 @@ static inline __u16 rta_getattr_u16(const struct rtattr *rta) { return *(__u16 *)RTA_DATA(rta); } +static inline __be16 rta_getattr_be16(const struct rtattr *rta) +{ + return ntohs(rta_getattr_u16(rta)); +} static inline __u32 rta_getattr_u32(const struct rtattr *rta) { return *(__u32 *)RTA_DATA(rta); } +static inline __be32 rta_getattr_be32(const struct rtattr *rta) +{ + return ntohl(rta_getattr_u32(rta)); +} static inline __u64 rta_getattr_u64(const struct rtattr *rta) { __u64 tmp; diff --git a/ip/iplink_geneve.c b/ip/iplink_geneve.c index 3bfba91c644c..1e6669d07d60 100644 --- a/ip/iplink_geneve.c +++ b/ip/iplink_geneve.c @@ -234,7 +234,7 @@ static void geneve_print_opt(struct link_util *lu, FILE *f, struct rtattr *tb[]) if (tb[IFLA_GENEVE_PORT]) fprintf(f, "dstport %u ", - ntohs(rta_getattr_u16(tb[IFLA_GENEVE_PORT]))); + rta_getattr_be16(tb[IFLA_GENEVE_PORT])); if (tb[IFLA_GENEVE_COLLECT_METADATA]) fputs("external ", f); diff --git a/ip/iplink_vxlan.c b/ip/iplink_vxlan.c index 93af979a1e97..6d02bb47b2f0 100644 --- a/ip/iplink_vxlan.c +++ b/ip/iplink_vxlan.c @@ -413,7 +413,7 @@ static void vxlan_print_opt(struct link_util *lu, FILE *f, struct rtattr *tb[]) if (tb[IFLA_VXLAN_PORT]) fprintf(f, "dstport %u ", - ntohs(rta_getattr_u16(tb[IFLA_VXLAN_PORT]))); + rta_getattr_be16(tb[IFLA_VXLAN_PORT])); if (tb[IFLA_VXLAN_LEARNING] && !rta_getattr_u8(tb[IFLA_VXLAN_LEARNING]))