Hi Dave: I was bored so here is a patch that optimises away a branch in IP_ECN_set_ce. Signed-off-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Cheers, -- Visit Openswan at http://www.openswan.org/ Em
Em Qua 08 Set 2004 21:36, YOSHIFUJI Hideaki / 吉藤英明 escreveu: Yes, that is the only case, like I learned in my early janitor days, using the __constant_ variations is not needed and makes the code nee
Hi Dave: I was bored so here is a patch that optimises away a branch in IP_ECN_set_ce. Signed-off-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Cheers, -- Visit Openswan at http://www.openswan.org/ Em
Author: Arnaldo Carvalho de Melo <acme@xxxxxxxxxxxxxxxx>
Date: Wed, 8 Sep 2004 21:53:18 -0300
Em Qua 08 Set 2004 21:36, YOSHIFUJI Hideaki / escreveu: Yes, that is the only case, like I learned in my early janitor days, using the __constant_ variations is not needed and makes the code needless
Good point. The second htons is less optimal than the shift because htons doesn't know that ecn is only a byte. So it'll end up doing a full swap. Signed-off-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxx
Sorry, you're right. Here is an updated patch. Signed-off-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <herbert@xxxx