Re: [ANNOUNCE 2/6] Linux-iSCSI High-Performance Initiator

2005-04-12 Thread Dmitry Yusupov
On Mon, 2005-04-11 at 22:35 -0700, Greg KH wrote: > On Mon, Apr 11, 2005 at 08:24:08PM -0700, Alex Aizman wrote: > > +typedef uint64_t iscsi_snx_t; /* iSCSI Data-Path session > > handle */ > > +typedef uint64_t iscsi_cnx_t; /* iSCSI Data-Path connection > > handle */ >

Re: [ANNOUNCE 2/6] Linux-iSCSI High-Performance Initiator

2005-04-12 Thread Christoph Hellwig
On Tue, Apr 12, 2005 at 12:45:14AM -0700, Greg KH wrote: > Um, why? We've been down this road before, and for types that cross the > boundry, we _must_ use the __ version of the kernel types, not the > uint32_t stuff. That's total bullshit. C99 types just work in both the kernel and userland, wh

Re: [ANNOUNCE 2/6] Linux-iSCSI High-Performance Initiator

2005-04-12 Thread Greg KH
On Tue, Apr 12, 2005 at 08:27:33AM +0100, Christoph Hellwig wrote: > On Mon, Apr 11, 2005 at 10:36:51PM -0700, Greg KH wrote: > > On Mon, Apr 11, 2005 at 08:24:08PM -0700, Alex Aizman wrote: > > > Common header files: > > > - iscsi_ifev.h (user/kernel events). > > > > T

Re: [ANNOUNCE 2/6] Linux-iSCSI High-Performance Initiator

2005-04-12 Thread Christoph Hellwig
On Mon, Apr 11, 2005 at 10:36:51PM -0700, Greg KH wrote: > On Mon, Apr 11, 2005 at 08:24:08PM -0700, Alex Aizman wrote: > > Common header files: > > - iscsi_ifev.h (user/kernel events). > > These structures cross the user/kernel boundry? If so, they _must_ use > the __

Re: [ANNOUNCE 2/6] Linux-iSCSI High-Performance Initiator

2005-04-11 Thread Greg KH
On Mon, Apr 11, 2005 at 08:24:08PM -0700, Alex Aizman wrote: > Common header files: > - iscsi_ifev.h (user/kernel events). These structures cross the user/kernel boundry? If so, they _must_ use the __u32 and friends types, not the horrible uint32_t mess... thanks, gr

Re: [ANNOUNCE 2/6] Linux-iSCSI High-Performance Initiator

2005-04-11 Thread Greg KH
> +struct iscsi_hdr { > + uint8_t opcode; > + uint8_t flags; /* Final bit */ > + uint8_t rsvd2[2]; > + uint8_t hlength;/* AHSs total length */ > + uint8_t dlength[3]; /* Data length */ > + uint8_t lun[8]; >

Re: [ANNOUNCE 2/6] Linux-iSCSI High-Performance Initiator

2005-04-11 Thread Greg KH
On Mon, Apr 11, 2005 at 08:24:08PM -0700, Alex Aizman wrote: > +typedef uint64_t iscsi_snx_t;/* iSCSI Data-Path session > handle */ > +typedef uint64_t iscsi_cnx_t;/* iSCSI Data-Path connection > handle */ Do you really have to create a new typedef? Please recons

[ANNOUNCE 2/6] Linux-iSCSI High-Performance Initiator

2005-04-11 Thread Alex Aizman
Common header files: - iscsi_ifev.h (user/kernel events). - iscsi_if.h (user/kernel #defines); - iscsi_iftrans.h (iscsi transport interface); - iscsi_proto.h (RFC3720 #defines and types). Signed-off-by: Alex Aizman