| To: | Werner Almesberger <wa@xxxxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] arp_queue: serializing unlink + kfree_skb |
| From: | Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> |
| Date: | Thu, 10 Feb 2005 15:56:47 +1100 |
| Cc: | "David S. Miller" <davem@xxxxxxxxxxxxx>, anton@xxxxxxxxx, okir@xxxxxxx, netdev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx |
| In-reply-to: | <20050210012304.E25338@almesberger.net> |
| References: | <20050131102920.GC4170@suse.de> <E1CvZo6-0001Bz-00@gondolin.me.apana.org.au> <20050203142705.GA11318@krispykreme.ozlabs.ibm.com> <20050203150821.2321130b.davem@davemloft.net> <20050204113305.GA12764@gondor.apana.org.au> <20050204154855.79340cdb.davem@davemloft.net> <20050204222428.1a13a482.davem@davemloft.net> <20050210012304.E25338@almesberger.net> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.5.6+20040722i |
On Thu, Feb 10, 2005 at 01:23:04AM -0300, Werner Almesberger wrote: > > What happens if the operation could return a value, but the user > ignores it ? E.g. if I don't like smp_mb__*, could I just use > > atomic_inc_and_test(foo); > > instead of > > smp_mb__before_atomic_inc(); > atomic_inc(foo); > smp_mb__after_atomic_dec(); Yes you can. > ? If yes, is this a good idea ? Dave mentioned that on sparc64, atomic_inc_and_test is much more expensive than the second variant. 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 |
| Previous by Date: | Re: [PATCH] ieee80211 subsystem, Randy.Dunlap |
|---|---|
| Next by Date: | [PATCH] Use TASK_COMM_LEN macro., YOSHIFUJI Hideaki / 吉藤英明 |
| Previous by Thread: | Re: [PATCH] arp_queue: serializing unlink + kfree_skb, Werner Almesberger |
| Next by Thread: | Re: [PATCH] arp_queue: serializing unlink + kfree_skb, David S. Miller |
| Indexes: | [Date] [Thread] [Top] [All Lists] |