Path: csiph.com!eternal-september.org!feeder.eternal-september.org!aioe.org!bofh.it!news.nic.it!robomod From: Amitkumar Karwar Newsgroups: linux.kernel Subject: RE: [PATCH 05/12] mwifiex: avoid gettimeofday in ba_threshold setting Date: Fri, 09 Oct 2015 13:40:02 +0200 Message-ID: References: X-Original-To: Arnd Bergmann , "netdev@vger.kernel.org" Thread-Topic: [PATCH 05/12] mwifiex: avoid gettimeofday in ba_threshold setting Thread-Index: AQHQ+3Lzb3Cp6qOL7Uu4GA8AWDEwLJ5jFitg Accept-Language: en-US Content-Language: en-US X-Ms-Exchange-Transport-Fromentityheader: Hosted X-Originating-IP: [10.93.176.43] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2015-10-09_10:,, signatures=0 X-Proofpoint-Spam-Details: rule=inbound_notspam policy=inbound score=0 spamscore=0 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1507310000 definitions=main-1510090144 Sender: robomod@news.nic.it List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Approved: robomod@news.nic.it Lines: 84 Organization: linux.* mail to news gateway X-Original-Cc: "y2038@lists.linaro.org" , "linux-kernel@vger.kernel.org" , "David S. Miller" , Nishant Sarmukadam , Kalle Valo , "linux-wireless@vger.kernel.org" X-Original-Date: Fri, 9 Oct 2015 11:35:47 +0000 X-Original-Message-ID: <4d9d7f2dc1a44e00b9ce74660e306640@SC-EXCH04.marvell.com> X-Original-References: <1443612402-3000775-1-git-send-email-arnd@arndb.de> <1443612402-3000775-6-git-send-email-arnd@arndb.de> X-Original-Sender: linux-kernel-owner@vger.kernel.org Xref: csiph.com linux.kernel:1243276 Hi Arnd, > From: Arnd Bergmann [mailto:arnd@arndb.de] > Sent: Wednesday, September 30, 2015 4:57 PM > To: netdev@vger.kernel.org > Cc: y2038@lists.linaro.org; linux-kernel@vger.kernel.org; David S. > Miller; Arnd Bergmann; Amitkumar Karwar; Nishant Sarmukadam; Kalle Valo; > linux-wireless@vger.kernel.org > Subject: [PATCH 05/12] mwifiex: avoid gettimeofday in ba_threshold > setting > > mwifiex_get_random_ba_threshold() uses a complex homegrown > implementation to generate a pseudo-random number from the current time > as returned from do_gettimeofday(). > > This currently requires two 32-bit divisions plus a couple of other > computations that are eventually discarded as only eight bits of the > microsecond portion are used at all. > > We could replace this with a call to get_random_bytes(), but that might > drain the entropy pool too fast if this is called for each packet. > > Instead, this patch converts it to use ktime_get_ns(), which is a bit > faster than do_gettimeofday(), and then uses a similar algorithm as > before, but in a way that takes both the nanosecond and second portion > into account for slightly-more-but-still-not-very-random > pseudorandom number. > > Signed-off-by: Arnd Bergmann > Cc: Amitkumar Karwar > Cc: Nishant Sarmukadam > Cc: Kalle Valo > Cc: linux-wireless@vger.kernel.org > --- > drivers/net/wireless/mwifiex/wmm.c | 15 ++++----------- > 1 file changed, 4 insertions(+), 11 deletions(-) > > diff --git a/drivers/net/wireless/mwifiex/wmm.c > b/drivers/net/wireless/mwifiex/wmm.c > index 173d3663c2e0..878d358063dc 100644 > --- a/drivers/net/wireless/mwifiex/wmm.c > +++ b/drivers/net/wireless/mwifiex/wmm.c > @@ -117,22 +117,15 @@ mwifiex_wmm_allocate_ralist_node(struct > mwifiex_adapter *adapter, const u8 *ra) > */ > static u8 mwifiex_get_random_ba_threshold(void) > { > - u32 sec, usec; > - struct timeval ba_tstamp; > - u8 ba_threshold; > - > + u64 ns; > /* setup ba_packet_threshold here random number between > * [BA_SETUP_PACKET_OFFSET, > * BA_SETUP_PACKET_OFFSET+BA_SETUP_MAX_PACKET_THRESHOLD-1] > */ > + ns = ktime_get_ns(); > + ns += (ns >> 32) + (ns >> 16); > > - do_gettimeofday(&ba_tstamp); > - sec = (ba_tstamp.tv_sec & 0xFFFF) + (ba_tstamp.tv_sec >> 16); > - usec = (ba_tstamp.tv_usec & 0xFFFF) + (ba_tstamp.tv_usec >> 16); > - ba_threshold = (((sec << 16) + usec) % > BA_SETUP_MAX_PACKET_THRESHOLD) > - + BA_SETUP_PACKET_OFFSET; > - > - return ba_threshold; > + return ((u8)ns % BA_SETUP_MAX_PACKET_THRESHOLD) + > +BA_SETUP_PACKET_OFFSET; > } > > /* > -- Looks fine to me. Acked-by: Amitkumar Karwar Regards, Amitkumar -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/