netdev
[Top] [All Lists]

Re: [RFC 2.6.10 1/22] xfrm: Add direction information to xfrm_state

To: Dave Dillow <dave@xxxxxxxxxxxxxx>
Subject: Re: [RFC 2.6.10 1/22] xfrm: Add direction information to xfrm_state
From: Jan-Benedict Glaw <jbglaw@xxxxxxxxxx>
Date: Thu, 30 Dec 2004 17:36:17 +0100
Cc: netdev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <1104423409.23254.9.camel@dillow.idleaire.com>
Mail-followup-to: Dave Dillow <dave@xxxxxxxxxxxxxx>, netdev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
References: <20041230035000.01@ori.thedillows.org> <20041230035000.10@ori.thedillows.org> <20041230094839.GX2460@lug-owl.de> <1104423409.23254.9.camel@dillow.idleaire.com>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.5.6+20040907i
On Thu, 2004-12-30 11:16:49 -0500, Dave Dillow <dave@xxxxxxxxxxxxxx>
wrote in message <1104423409.23254.9.camel@xxxxxxxxxxxxxxxxxxx>:
> On Thu, 2004-12-30 at 04:48, Jan-Benedict Glaw wrote:
> > On Thu, 2004-12-30 03:48:34 -0500, David Dillow <dave@xxxxxxxxxxxxxx>
> > wrote in message <20041230035000.10@xxxxxxxxxxxxxxxxxx>:
> > > +enum {
> > > + XFRM_STATE_DIR_UNKNOWN,
> > > + XFRM_STATE_DIR_IN,
> > > + XFRM_STATE_DIR_OUT,
> > > +};
> > 
> > Any specific reason to first define such a nice enum and then using int
> > in the struct?
> 
> Just following the current style in net/xfrm.h, see xfrm_state.km.state
> and XFRM_STATE_*.

Hmmm... Maybe I'd prepare patches then :)

MfG, JBG

-- 
Jan-Benedict Glaw       jbglaw@xxxxxxxxxx    . +49-172-7608481             _ O _
"Eine Freie Meinung in  einem Freien Kopf    | Gegen Zensur | Gegen Krieg  _ _ O
 fuer einen Freien Staat voll Freier BÃrger" | im Internet! |   im Irak!   O O 
O
ret = do_actions((curr | FREE_SPEECH) & ~(NEW_COPYRIGHT_LAW | DRM | TCPA));

Attachment: signature.asc
Description: Digital signature

<Prev in Thread] Current Thread [Next in Thread>