lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20100321004659.GA5895@gondor.apana.org.au> Date: Sun, 21 Mar 2010 08:46:59 +0800 From: Herbert Xu <herbert@...dor.apana.org.au> To: Timo Teräs <timo.teras@....fi> Cc: netdev@...r.kernel.org, "David S. Miller" <davem@...emloft.net> Subject: Re: [PATCH] xfrm: cache bundle lookup results in flow cache On Sat, Mar 20, 2010 at 06:26:00PM +0200, Timo Teräs wrote: > > So should go ahead and: > 1. modify flow cache to be more generic (have virtual put and get > for each object; and remove the atomic_t pointer) > 2. modify flow cache to have slow and fast resolvers so we can > copy with the current sleeping requirement I don't think we need either of these. To support the sleep requirement, just return -EAGAIN from the resolver when the template can't be resolved. Then the caller of flow_cache_lookup can sleep as it does now. It simply has to repeat the flow cache lookup afterwards. > 3. cache bundles instead of policies for outgoing stuff > 4. kill find_bundle and just instantiate new ones if we get cache > miss > 5. put all bundles to global hlist (since only place that walks > through them is gc, and stale bundle can be dst_free'd right > away); use genid's for policy to flush old bundles > 6. dst_free and unlink bundle immediately if it's found to be stale Sounds good. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <herbert@...dor.apana.org.au> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists