Message ID | 1528971137-432-1-git-send-email-omer.efrat@tandemg.com (mailing list archive) |
---|---|
State | Superseded, archived |
Delegated to: | Simon Wunderlich |
Headers |
Return-Path: <b.a.t.m.a.n-bounces@lists.open-mesh.org> X-Original-To: patchwork@open-mesh.org Delivered-To: patchwork@open-mesh.org Received: from open-mesh.org (localhost [IPv6:::1]) by open-mesh.org (Postfix) with ESMTP id C6FFC81164; Thu, 14 Jun 2018 12:18:53 +0200 (CEST) Authentication-Results: open-mesh.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=tandemg.com header.i=@tandemg.com header.b="SD4l14T9"; dkim-atps=neutral Received-SPF: Pass (mailfrom) identity=mailfrom; client-ip=2a01:111:f400:fe0c::707; helo=eur04-db3-obe.outbound.protection.outlook.com; envelope-from=omer.efrat@tandemg.com; receiver=<UNKNOWN> Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-db3eur04on0707.outbound.protection.outlook.com [IPv6:2a01:111:f400:fe0c::707]) by open-mesh.org (Postfix) with ESMTPS id 96A17805EB for <b.a.t.m.a.n@lists.open-mesh.org>; Thu, 14 Jun 2018 12:12:23 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tandemg.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=WXq/oymOcsekHtPc/G0FfRx+YgNvx09uFb++xRFJ614=; b=SD4l14T9CXnHlYPUkxjL9a5S60bmmZEuR+HK1aPEOMgEPdaj0AB8Vd74Uw/b3EGNeReA56eq0P0ZjICxNaNFjgrq4AqricDAOaF+MRlitjfX1XbV91ARENuBaLOgtpq8MdehgJM+6EMa+1ueHiXFUfpL0cshzXkHMZpWFvOIeyg= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=omer.efrat@tandemg.com; Received: from omer-laptop.tandemg.local (37.142.120.98) by AM0PR02MB3619.eurprd02.prod.outlook.com (2603:10a6:208:3f::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.863.16; Thu, 14 Jun 2018 10:12:21 +0000 From: Omer Efrat <omer.efrat@tandemg.com> To: linux-wireless@vger.kernel.org Date: Thu, 14 Jun 2018 13:12:17 +0300 Message-Id: <1528971137-432-1-git-send-email-omer.efrat@tandemg.com> X-Mailer: git-send-email 2.7.4 MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [37.142.120.98] X-ClientProxiedBy: CWLP265CA0136.GBRP265.PROD.OUTLOOK.COM (2603:10a6:401:53::28) To AM0PR02MB3619.eurprd02.prod.outlook.com (2603:10a6:208:3f::20) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 1e469763-1cdb-4796-0e8e-08d5d1df51c1 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(7020095)(4652020)(5600026)(711020)(4534165)(4627221)(201703031133081)(201702281549075)(2017052603328)(7153060)(7193020); SRVR:AM0PR02MB3619; X-Microsoft-Exchange-Diagnostics: 1; AM0PR02MB3619; 3:PddUOofZ1OpYkqm8trRrFqSzTeDWj8rhqBcNMZHxwz6BpjD6BOG6HeBsKUQ9QKLg6DCiWWMmuTAk9VHGuQrBPNV4suG0KCpEuUGrs08m77B4PJIgaNYezSIQ372r/16SC+kFbyH7+DNqx0NOP5HEQOQ/nB127hsc4N0bIH30rodq6ludlCqtK6BiSobN0FR5tA/4Wp5x8BN9d2Q7FhjmUB6cwxgWFgxNyOFa1PxuOeOHoRS/0ZyysC9KQFinX01j; 25:rzqmW6FBpZabWZReIVQtSKoSW37tLW3uJxuHCnIOrBdXqOxmvRbx72ufIM3cYjPAkYFpC9e+Y7bh7CKowypQUtNS55alos5D6x7eY/gFyXRLTvNo2Ged35Htdch49AsZTh9dxQblmk1WsguBcanFx4nsj3FCdzgfDqiU654VFrx6JwAQ+bBzueP9kMxF9tRW9fhIj/3uNpKFxHL3AY+YtLrZo76NWZldwJlZVDTIUpy7gNvhGDH5hE7Bz4RT3iDrZEgzWDbV9CfoR2+CZQifxIhK1nY09tX3Vrz95L+flmhbxuS74XyayFFABm8rrweRixxwH6PMX5p4j8kO31Goig==; 31:7CdILcoZ2bmt1gycUHiQUIJGDbxGroqzSuXWTt6aMH888Kb2LnuvB4quZCpJZPCpc2mEyCxSvIMoiAAhXBv/yaIC0rK6mBhdGPK2BhDblKIiZcq1zu8h3jXgI5u20xLqIDIG3Ys/GiJGrMtsvPk5s1tLWbCYB9u7kUOksD/ky2wBY9G7jp+3t7r6Joy3aRDykPj4Gq5omOfO5sXyxM7gCFgUyDUh2FMQY/wXqg0p+/c= X-MS-TrafficTypeDiagnostic: AM0PR02MB3619: X-Microsoft-Exchange-Diagnostics: 1; AM0PR02MB3619; 20:BBrwn/XhlURoGK6ctYdw7fs6fo1ubQCAiD1PCOyqjEJNnmR/iLsGx6Wpm7leNzjz6zNLF7EJMOl4sgwaQ8TmEvhHPmRpzMFJL4ZIEzZ08p8cEk5cNLAMPN35Hb3qSqBCTQSK+Fnl5ETZ7ZIwWVimCGXr5pBJ1Wdc7VaGWxI2FeB+djXjf5mBxFTT3KERMcrMQYzLF3laAaxUh/mcUZvzj1PEWrPTXy6p1lx7inyANXWBzkZd68oVtj50R5m2CuZX6gixt1W1cxAFC2KXN5jZXNH5V7OKx29NqgxWAANf/05YrzFyLBta9muEW7sTcgTfqP4l93MqdnQVrnudBnC//w==; 4:FO2/4PnCYjYQ/wqVO7/QABN/Vh5EcSfwD+KHb3TGo5hjZqHL0cs6z4LIqes/sxQZdwkTRTjHN6yTYL1LcGEoj2GoERuXh1tx2qnNT1cdM1GneIHj5jpXVDFfm2tkCZB7svAZ290L/w5kMq1so/yIXAEiXKSRbeykrvxd2OryaD9sTgQqDM+XYdnqmBbGc36O32v+JFUM8fxM/zeG0CVWppV5pymWXPps0aORHL2Iz8QvVdpLJq0a18t/VLy+TqYhzjZuFCCeyVmG2I3mrkYSHA== X-Microsoft-Antispam-PRVS: <AM0PR02MB3619C9F50F2EA102AAC8CE689A7D0@AM0PR02MB3619.eurprd02.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:; X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040522)(2401047)(5005006)(8121501046)(10201501046)(3231254)(944501410)(52105095)(93006095)(93001095)(3002001)(149027)(150027)(6041310)(20161123560045)(20161123564045)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123558120)(6072148)(201708071742011)(7699016); SRVR:AM0PR02MB3619; BCL:0; PCL:0; RULEID:; SRVR:AM0PR02MB3619; X-Forefront-PRVS: 0703B549E4 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10019020)(39840400004)(39380400002)(396003)(346002)(366004)(376002)(189003)(199004)(53936002)(305945005)(4326008)(25786009)(106356001)(2361001)(66066001)(6486002)(105586002)(956004)(476003)(16586007)(486006)(107886003)(5660300001)(2616005)(6512007)(7736002)(316002)(44832011)(36756003)(47776003)(2351001)(68736007)(59450400001)(6506007)(386003)(8936002)(50226002)(8676002)(81156014)(2906002)(81166006)(478600001)(51416003)(52116002)(26005)(50466002)(186003)(6666003)(16526019)(48376002)(86362001)(6916009)(6116002)(97736004)(3846002)(15760500003); DIR:OUT; SFP:1102; SCL:1; SRVR:AM0PR02MB3619; H:omer-laptop.tandemg.local; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; Received-SPF: None (protection.outlook.com: tandemg.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; AM0PR02MB3619; 23:ulj/SqGFW5zYB1Tm8MkZUnVbGnMFLM7E8UHA03XUW?= 7fDxp2/xcJ85hZ8s36ZNv7AvcWX000KIIQCdsEO+XLLgk+nyR+LfIB2kY3/6mY4VRik07brwG6N/uJRf/YtOv0i/tmdbICPA5TQf3c9/bh+obr94WgHiIYmjgQvth/HCpi4D5pQ1VEIjG0Chwyy0IJsuVKs4RHmV1yW6CCInd8AVjifJaOc9QdRF2T19MDAZg5qPuERBpmwoBubEY1kBC5vVfQFXWu+HOVNDp6F8nZiA9GbPRYKpM8+ISZeSmVhbRBl6wB3OdDsaltLnh3oA7/O0RUrAZ0CZpSFIbY5vmAe8+7j7nRevJrvyYgu2FYobL23hXFU4qHCJd4gtCgzZGFkCuP0zpbvDF1Pd/9fDWfDQlk+JfOOVshJUtfm1/2AXpWBumrNWgHAH0Q44jCrE9ysM2LoWYQu+d7Ww+zgKhKOfbsRF80UCOv8ANMBBqd9uLx2tInsrLEEMC2IuOe/ChXLFKodgxxxX9nHknCOX+nh7USOVFiPPuSNpu29juvccxrO/D+LEd+K+lSCWHABx43y0gDcRlLggyMUEkJb0AwTID2qU6GObf7LHqScxyQGBXrMhE7r62HGun1vAdRPbsvhBMkxjjdyEW9nrDMtUXh/rJositIvaH34NiBqgzdeLGI1gCKp2v/LE4RnK7J9yv40+SnlbL6joEvGHH1z7qEspo9pAza+zbHT+Sl8lm7Lsmfco0Thf5vfsFmIJMt2zSSoxXcBOdrnYY8AbTm3Az3p/XwAeqJdCpheeGo40DmKghiJWk1MzZpKcyNeE5AIRmp7ToR7bdxMZfeoMCFD9R0L2mvlGPElLNWn9GXYVzAPoOga1N0kR1bQeGCyES/ENPZNVadnKa0j1SunzMIJSQf0UtzWybhMrv/b7MfpoUcKDrVXrv3vIetnuSY/puDSurmNKJurJmwimijSxLRbHIgdDAvp6x9gOfiiGfkZ3konjLZmI+Xc1JZM1VluYTygfTCxrZ+C8s5vJ/mUC1Fe+wY6+qMrJBWhFIlqAt4QReAik3ybMbkmC8v8/63NHvK0UgSHb1Low5LnG8eWkUFid49XLRRk3cHbvB+MZZSCZIZ9fZ8VjPZOBWL1NAf1h2AbgTgtGSZL9ihb8YJwCzIKVum928eLeZUNl8uQsv0BXyCXonUdyfufmiIn43cE78UFqktqoEHG0Yh3QRi6S2zzuwP0XFOgok8c1wppPwqDlHgsFSY= X-Microsoft-Antispam-Message-Info: EiPogHqsVHdV3LjxcTNKXECmBo7RvWsh37yCQhmZ3gpDsU7xHOuIc+RS9Gf7lsrwOSOoE3Yczr83GSxycclCULVUJ7NMkeBvDEJDhJqn5YsN6fzpl3HNH5JQvyDaddLl10zRWrm6VBXu9uyfrA62QehO/eTa7UPNSY5rb1wWYTqGLY+dcsW7U0jtALWw7uzY X-Microsoft-Exchange-Diagnostics: 1; AM0PR02MB3619; 6:hypchZ5/Is4V2GgQtQop6pCikk4P9yCqik1dmTfxM+L9JXnad4hQyf4xBhreriBesFSoEXURPfnphGhtfvoePiOzMmnP9hCuUwgA4mUlrNhZ1vtLcfvhA26+iW+vNEpykqeL4ZsTHNVRSUcqUbaC/Se0+ID/D3CqayHSHkg4YRYLpRn7cgp6TuHLnvuR4sTiBAnX7vweg9q2DjzCU6bTjgB0bJ46A/G1OZ+Cx3xsRzn9j5XBe8JrEY42Dl4gdPTxSjR0EF25rJXDxWqoMtcbY2KbItlRlq3GNi5px5PZnFvyFEva+fZHT0Et/cSdS6N2zJY1m/RwCupZI/OV7mFSstwgSjrLS6M+84Z6Ju6JZiIUAeAUcGcozpV528S8NHBQS+ALkQz8LfPSsPJic9OR/rLTa8wq0o5HECyhHgBsDikRQyzi8mGROEPznHE3NRKDFAgURi2lbndn8k1pm4qZdg==; 5:xToDO7fAD8jj51Oy0NT2huWewIOUDKu5j5hb8pibWyiEX4lrsCggL8iA7x40HYVlX9uhR2YEls3mL7sjIA1ygjxyc9Pw7AG5PIJyWtmmO2xzr2D93Ltj0pqoy/Z/uNxopQSPcG9xMsQDInbsBQMMPYGo3VDHsY5+YT+TIYjqOE4=; 24:H/tDoBEdH7bgL+5C94CvIynG5NRU5dks3L7q2PHVQEewpeuzBQ/GSbe7xGxWH2kGkcC3NoHLJPbt52mAPprcXDjbPGeh1+vN/9RETbdc+Ks= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; AM0PR02MB3619; 7:ZxLKBTcXSJoSOtqXpOpB+4sRLFnoJqog9ceAVP7Ye689m6DyEbhq8PsNnPJL2jb+q+2STGrOerl1X7/os8N+Q5WrfWa8SP5vFyAzMdscjR2n0t5nYBYhPv8N9XCgtHAeaNzAktV4lYrCrYOWBCzkcyQb5RLonnw26Q8Wk9qR6pVpNBgOQYREnlwoKfnbI0gVHZlUsPpkyt8GLiPw0cB5fvDnHOhqVoTR1/59l7zAEQ6JX6G+X14aBmbF6novpZKw X-OriginatorOrg: tandemg.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Jun 2018 10:12:21.7312 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 1e469763-1cdb-4796-0e8e-08d5d1df51c1 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: d690b55a-f04a-454b-9f62-fb1e25467a25 X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR02MB3619 X-Mailman-Approved-At: Thu, 14 Jun 2018 12:18:51 +0200 Subject: [B.A.T.M.A.N.] [PATCH v2 3/5] batman: use BIT_ULL for NL80211_STA_INFO_* attribute types X-BeenThere: b.a.t.m.a.n@lists.open-mesh.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: The list for a Better Approach To Mobile Ad-hoc Networking <b.a.t.m.a.n.lists.open-mesh.org> List-Unsubscribe: <https://lists.open-mesh.org/mm/options/b.a.t.m.a.n>, <mailto:b.a.t.m.a.n-request@lists.open-mesh.org?subject=unsubscribe> List-Archive: <http://lists.open-mesh.org/pipermail/b.a.t.m.a.n/> List-Post: <mailto:b.a.t.m.a.n@lists.open-mesh.org> List-Help: <mailto:b.a.t.m.a.n-request@lists.open-mesh.org?subject=help> List-Subscribe: <https://lists.open-mesh.org/mm/listinfo/b.a.t.m.a.n>, <mailto:b.a.t.m.a.n-request@lists.open-mesh.org?subject=subscribe> Reply-To: The list for a Better Approach To Mobile Ad-hoc Networking <b.a.t.m.a.n@lists.open-mesh.org> Cc: netdev@vger.kernel.org, b.a.t.m.a.n@lists.open-mesh.org, Omer Efrat <omer.efrat@tandemg.com> Errors-To: b.a.t.m.a.n-bounces@lists.open-mesh.org Sender: "B.A.T.M.A.N" <b.a.t.m.a.n-bounces@lists.open-mesh.org> |
Series |
None
|
|
Commit Message
Omer Efrat
June 14, 2018, 10:12 a.m. UTC
Since 'filled' member in station_info changed to u64, BIT_ULL macro
should be used with NL80211_STA_INFO_* attribute types instead of BIT.
The BIT macro uses unsigned long type which some architectures handle as 32bit
and this results in compilation warnings such as:
net/mac80211/sta_info.c:2223:2: warning: left shift count >= width of type
sinfo->filled |= BIT(NL80211_STA_INFO_TID_STATS);
^
Signed-off-by: Omer Efrat <omer.efrat@tandemg.com>
---
net/batman-adv/bat_v_elp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
On Donnerstag, 14. Juni 2018 13:12:17 CEST Omer Efrat wrote: > Since 'filled' member in station_info changed to u64, BIT_ULL macro > should be used with NL80211_STA_INFO_* attribute types instead of BIT. > > The BIT macro uses unsigned long type which some architectures handle as 32bit > and this results in compilation warnings such as: > > net/mac80211/sta_info.c:2223:2: warning: left shift count >= width of type > sinfo->filled |= BIT(NL80211_STA_INFO_TID_STATS); > ^ > > Signed-off-by: Omer Efrat <omer.efrat@tandemg.com> > --- > net/batman-adv/bat_v_elp.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) It is called "batman-adv" and not "batman". And when (as in commit) did it change to 64 bit? Shouldn't there be a "Fixed: " line to know which kernels are affected (especially for the stable kernel developers). Kind regards, Sven
Hi, here are the infos which were missing and which should be included in the commit message > > Since 'filled' member in station_info changed to u64 in commit 739960f128e5 ("cfg80211/nl80211: Add support for NL80211_STA_INFO_RX_DURATION") [...] Fixes: d62890885efb ("batman-adv: Accept only filled wifi station info") > > Signed-off-by: Omer Efrat <omer.efrat@tandemg.com> Kind regards, Sven
On Thu, 2018-06-14 at 12:53 +0200, Sven Eckelmann wrote: > Hi, > > here are the infos which were missing and which should be included in the > commit message > > > > Since 'filled' member in station_info changed to u64 > > in commit 739960f128e5 ("cfg80211/nl80211: Add support for > NL80211_STA_INFO_RX_DURATION") Yeah, which actually means this patch isn't needed? BIT(NL80211_STA_INFO_EXPECTED_THROUGHPUT) is fine since NL80211_STA_INFO_EXPECTED_THROUGHPUT is actually == 27. johannes
On Donnerstag, 14. Juni 2018 13:05:16 CEST Johannes Berg wrote: [...] > > in commit 739960f128e5 ("cfg80211/nl80211: Add support for > > NL80211_STA_INFO_RX_DURATION") > > Yeah, which actually means this patch isn't needed? > > BIT(NL80211_STA_INFO_EXPECTED_THROUGHPUT) is fine since > NL80211_STA_INFO_EXPECTED_THROUGHPUT is actually == 27. Hadn't verified this before but this would make sense. So no fixes here - just some "cleanup" patch to make these tests more consistent. Thanks for checking. @Omer: If you want it as cleanup patch then make it clear in the patch that the warning you've showed here is not actually not something which you will see in in the modified code. Kind regards, Sven
Sven Eckelmann wrote: >@Omer: If you want it as cleanup patch then make it clear in the patch that >the warning you've showed here is not actually not something which you will >see in in the modified code. I will send v3 as clean up patch. Omer Efrat.
diff --git a/net/batman-adv/bat_v_elp.c b/net/batman-adv/bat_v_elp.c index 71c20c1..71e6474 100644 --- a/net/batman-adv/bat_v_elp.c +++ b/net/batman-adv/bat_v_elp.c @@ -114,7 +114,7 @@ static u32 batadv_v_elp_get_throughput(struct batadv_hardif_neigh_node *neigh) } if (ret) goto default_throughput; - if (!(sinfo.filled & BIT(NL80211_STA_INFO_EXPECTED_THROUGHPUT))) + if (!(sinfo.filled & BIT_ULL(NL80211_STA_INFO_EXPECTED_THROUGHPUT))) goto default_throughput; return sinfo.expected_throughput / 100;