| To: | jamal <hadi@xxxxxxxxxx> |
|---|---|
| Subject: | Re: patch: policy update by id |
| From: | Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> |
| Date: | Thu, 28 Apr 2005 12:07:54 +1000 |
| Cc: | netdev@xxxxxxxxxxx, "David S. Miller" <davem@xxxxxxxxxxxxx> |
| In-reply-to: | <1114653140.7663.36.camel@xxxxxxxxxxxxxxxxxxxxx> |
| References: | <1114602874.7670.4.camel@xxxxxxxxxxxxxxxxxxxxx> <1114604657.7670.22.camel@xxxxxxxxxxxxxxxxxxxxx> <1114604826.7670.24.camel@xxxxxxxxxxxxxxxxxxxxx> <20050427233924.GA22238@xxxxxxxxxxxxxxxxxxx> <1114650816.7663.13.camel@xxxxxxxxxxxxxxxxxxxxx> <20050428012135.GA22950@xxxxxxxxxxxxxxxxxxx> <20050428013014.GA23043@xxxxxxxxxxxxxxxxxxx> <1114653140.7663.36.camel@xxxxxxxxxxxxxxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.5.6+20040907i |
On Wed, Apr 27, 2005 at 09:52:20PM -0400, jamal wrote: > > > 1) There is only one policy with a given selector. > > 2) There is only one policy with a given index. > > Well, while snooping i was bothered as well. I am not sure i agree with > your #1 above ;-> > > 1) It would seem to me that the priority field is to be used > as a ambiguity resolver (thats what a gazillion other classification > schemes do). You know what, I actually agree with you :) But you'll need to convince Dave: http://www.uwsg.iu.edu/hypermail/linux/net/0305.3/0018.html However, this doesn't change the fact that you may need to delete two policies. > 2) index really oughta be unique across the SPD. > Current behavior: I can add several new rules with the same index. Not really. The kernel ignores the index supplied when you're adding them. Cheers, -- 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: patch: policy update by id, jamal |
|---|---|
| Next by Date: | patch2: del/get byid, jamal |
| Previous by Thread: | Re: patch: policy update by id, jamal |
| Next by Thread: | Re: patch: policy update by id, jamal |
| Indexes: | [Date] [Thread] [Top] [All Lists] |