| To: | jamal <hadi@xxxxxxxxxx> |
|---|---|
| Subject: | Re: RFC: IPSEC patch 0 for netlink events |
| From: | Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> |
| Date: | Thu, 31 Mar 2005 13:01:45 +1000 |
| Cc: | Patrick McHardy <kaber@xxxxxxxxx>, Masahide NAKAMURA <nakam@xxxxxxxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxxxxx>, netdev <netdev@xxxxxxxxxxx> |
| In-reply-to: | <20050331030036.GA27309@xxxxxxxxxxxxxxxxxxx> |
| References: | <1111864971.1092.904.camel@xxxxxxxxxxxxxxxx> <20050326194707.GA9872@xxxxxxxxxxxxxxxxxxx> <1111867875.1089.915.camel@xxxxxxxxxxxxxxxx> <20050327081848.GA13428@xxxxxxxxxxxxxxxxxxx> <1111950449.1089.938.camel@xxxxxxxxxxxxxxxx> <20050330004903.GA3399@xxxxxxxxxxxxxxxxxxx> <1112237759.1092.35.camel@xxxxxxxxxxxxxxxx> <20050331030036.GA27309@xxxxxxxxxxxxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.5.6+20040907i |
On Thu, Mar 31, 2005 at 01:00:36PM +1000, herbert wrote: > On Wed, Mar 30, 2005 at 09:55:59PM -0500, jamal wrote: > > > > Any objection to still calling it (new name) struct xfrm_sa_cb but > > with only one 32 bit value in it - namely data? > > This way i could add any other thing in the future in it without > > changing the function call paramaters. > > That's fine by me. Actually, could you rename it to km_state_cb? This information is only used for communication with the the km and it should be marked as such just like the other km_* identifiers. -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: RFC: IPSEC patch 0 for netlink events, Herbert Xu |
|---|---|
| Next by Date: | Re: [PATCH] netif_rx: receive path optimization, jamal |
| Previous by Thread: | Re: RFC: IPSEC patch 0 for netlink events, Herbert Xu |
| Next by Thread: | Re: RFC: IPSEC patch 0 for netlink events, jamal |
| Indexes: | [Date] [Thread] [Top] [All Lists] |