Sent from my iPad > On 30 dec. 2013, at 19:14, Julia Lawall <Julia.Lawall@xxxxxxx> wrote: > > From: Julia Lawall <Julia.Lawall@xxxxxxx> > > Ether_addr_equal_64bits is more efficient than ether_addr_equal, and can be > used when each argument is an array within a structure that contains at > least two bytes of data beyond the array. > > The structures involved are: > ieee80211_bar defined in include/linux/ieee80211.h and > rt2x00_bar_list_entry defined in drivers/net/wireless/rt2x00/rt2x00.h. > > This was done using Coccinelle (http://coccinelle.lip6.fr/). > > Signed-off-by: Julia Lawall <Julia.Lawall@xxxxxxx> Acked-by: Gertjan van Wingerde <gwingerde@xxxxxxxxx> > > --- > The semantic patch used is rather long and can be found in message 0 of > this patch series. > > drivers/net/wireless/rt2x00/rt2x00dev.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/rt2x00/rt2x00dev.c b/drivers/net/wireless/rt2x00/rt2x00dev.c > index 00c3fae..2bde672 100644 > --- a/drivers/net/wireless/rt2x00/rt2x00dev.c > +++ b/drivers/net/wireless/rt2x00/rt2x00dev.c > @@ -565,10 +565,10 @@ static void rt2x00lib_rxdone_check_ba(struct rt2x00_dev *rt2x00dev, > > #undef TID_CHECK > > - if (!ether_addr_equal(ba->ra, entry->ta)) > + if (!ether_addr_equal_64bits(ba->ra, entry->ta)) > continue; > > - if (!ether_addr_equal(ba->ta, entry->ra)) > + if (!ether_addr_equal_64bits(ba->ta, entry->ra)) > continue; > > /* Mark BAR since we received the according BA */ > -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html