| To: | herbert@xxxxxxxxxxxxxxxxxxx |
|---|---|
| Subject: | Re: [RFC][PATCH] identify in_dev_get rcu read-side critical sections |
| From: | Suzanne Wood <suzannew@xxxxxxxxxx> |
| Date: | Sat, 1 Oct 2005 11:37:14 -0700 (PDT) |
| Cc: | Robert.Olsson@xxxxxxxxxxx, davem@xxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, paulmck@xxxxxxxxxx, walpole@xxxxxxxxxx |
| Sender: | netdev-bounce@xxxxxxxxxxx |
Please excuse this restatement of an earlier concern. > From suzannew Sat Oct 1 11:00:28 2005 > With the spotlight leaving in_dev_get, we have the parallel question > of in_dev_put() and __in_dev_put() both defined with refcnt > decrement, but the preceding underscore may lend itself to an > inadvertant pairing and refcnt inaccuracy. Dave Miller already addressed this question in http://www.ussg.iu.edu/hypermail/linux/kernel/0509.3/0757.html > It may not be reasonable to rename __in_dev_put for its parallel definition > since its current usage is with __in_dev_get_rtnl() which does not increment > refcnt. But the following may be worth considering. > It is also probably good to retain the old __in_dev_get() and deprecate it. Thank you. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [RFC][PATCH] identify in_dev_get rcu read-side critical sections, Paul E. McKenney |
|---|---|
| Next by Date: | Re: [RFC][PATCH] identify in_dev_get rcu read-side critical sections, Herbert Xu |
| Previous by Thread: | Re: [RFC][PATCH] identify in_dev_get rcu read-side critical sections, Suzanne Wood |
| Next by Thread: | Re: [RFC][PATCH] identify in_dev_get rcu read-side critical sections, Herbert Xu |
| Indexes: | [Date] [Thread] [Top] [All Lists] |