Return-path: Received: from he.sipsolutions.net ([78.46.109.217]:35944 "EHLO sipsolutions.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754663Ab3AZXFD (ORCPT ); Sat, 26 Jan 2013 18:05:03 -0500 Message-ID: <1359241527.9086.5.camel@jlt4.sipsolutions.net> (sfid-20130127_000513_684797_DD38069F) Subject: Re: [PATCH v2] mac80211: dynamic short slot time for MBSSs From: Johannes Berg To: Thomas Pedersen Cc: linux-wireless@vger.kernel.org, devel@lists.open80211s.org Date: Sun, 27 Jan 2013 00:05:27 +0100 In-Reply-To: <1359241136.9086.0.camel@jlt4.sipsolutions.net> (sfid-20130126_235844_032457_426457DC) References: <1359232645-10563-1-git-send-email-thomas@cozybit.com> (sfid-20130126_213837_005675_90531EEB) <1359241136.9086.0.camel@jlt4.sipsolutions.net> (sfid-20130126_235844_032457_426457DC) Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Sat, 2013-01-26 at 23:58 +0100, Johannes Berg wrote: > On Sat, 2013-01-26 at 12:37 -0800, Thomas Pedersen wrote: > > > + u32 rates, changed = 0; > > > + for_each_set_bit(i, (unsigned long *)&rates, > > + sizeof(rates) * BITS_PER_BYTE) { > > this is broken on 64-bit architectures. Actually, probably only on big endian 64-bit architectures. johannes