- Subject: Re: [PATCH 2/5] drivers/net/ethernet/dec/tulip: Use standard __set_bit_le() function
- From: Akinobu Mita <akinobu.mita@xxxxxxxxx>
- Date: Wed, 13 Jun 2012 22:31:13 +0900
- Cc: Takuya Yoshikawa <yoshikawa.takuya@xxxxxxxxxxxxx>, akpm@xxxxxxxxxxxxxxxxxxxx, bhutchings@xxxxxxxxxxxxxx, grundler@xxxxxxxxxxxxxxxx, arnd@xxxxxxxx, benh@xxxxxxxxxxxxxxxxxxx, avi@xxxxxxxxxx, mtosatti@xxxxxxxxxx, linux-net-drivers@xxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-arch@xxxxxxxxxxxxxxx, kvm@xxxxxxxxxxxxxxx
- In-reply-to: <20120613214157.0a5179d5358ec7f1b2646606@gmail.com>
- References: <20120613130054.b5695621.yoshikawa.takuya@oss.ntt.co.jp> <20120613130304.0186fa9d.yoshikawa.takuya@oss.ntt.co.jp> <CAC5umyj582T+9GbOYDzOAanmMcVZ-Fdxr7i-8GWf3gTSmtAKUA@mail.gmail.com> <20120613214157.0a5179d5358ec7f1b2646606@gmail.com>
2012/6/13 Takuya Yoshikawa <takuya.yoshikawa@xxxxxxxxx>:
> On Wed, 13 Jun 2012 18:43:40 +0900
> Akinobu Mita <akinobu.mita@xxxxxxxxx> wrote:
>
>> Should this hash_table be converted from u16 hash_table[32] to
>> DECLARE_BITMAP(hash_table, 16 * 32) to ensure that it is aligned
>> on long-word boundary?
>
> I think hash_table is already long-word aligned because it is placed
> right after a pointer.
I recommend converting to proper bitmap. Because such an implicit
assumption is easily broken by someone touching this function.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[KVM ARM]
[KVM ia64]
[KVM ppc]
[Spice Development]
[Libvirt]
[Libvirt Users]
[Linux USB Devel]
[Video for Linux]
[Linux Audio Users]
[Photo]
[Yosemite News]
[Yosemite Photos]
[Linux Kernel]
[Linux SCSI]
[XFree86]