From mboxrd@z Thu Jan 1 00:00:00 1970 From: YOSHIFUJI Hideaki / =?iso-2022-jp?B?GyRCNUhGIzFRTEAbKEI=?= Subject: Re: [PATCH 2/3] add open iscsi netlink interface to iscsi transport class Date: Thu, 05 May 2005 20:55:30 +0900 (JST) Message-ID: <20050505.205530.107537707.yoshfuji@linux-ipv6.org> References: <20050505.185503.78606493.yoshfuji@linux-ipv6.org> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: michaelc@cs.wisc.edu, linux-scsi@vger.kernel.org, netdev@oss.sgi.com, linux-kernel@vger.kernel.org, yoshfuji@linux-ipv6.org Return-path: To: herbert@gondor.apana.org.au In-Reply-To: Sender: linux-scsi-owner@vger.kernel.org List-Id: netdev.vger.kernel.org In article (at Thu, 05 May 2005 21:39:47 +1000), Herbert Xu says: > YOSHIFUJI Hideaki / ???? wrote: > > In article <42798AC1.2010308@cs.wisc.edu> (at Wed, 04 May 2005 19:53:53 -0700), Mike Christie says: > > > >> +struct iscsi_uevent { > > : > >> +} __attribute__ ((aligned (sizeof(unsigned long)))); > > > > I think it'd be better to use sizeof(uint64_t) or something. > > Is there a benefit in aligning on 64-bit boundaries for 32-bit platforms? Well, this if for avoiding inconsistency between kernel and userspace. If you use unsigned long, sizeof(unsigned long) may be inconssitent between kernel and userland like this: kernel: sizeof(unsigned long) == 8 userspace: sizeof(unsigned long) == 4 --yoshfuji