* [PATCH v2] cnic: Replace rcu_dereference() with rcu_access_pointer()
@ 2014-08-17 10:12 Andreea-Cristina Bernat
2014-08-18 23:13 ` Michael Chan
2014-08-22 19:27 ` David Miller
0 siblings, 2 replies; 3+ messages in thread
From: Andreea-Cristina Bernat @ 2014-08-17 10:12 UTC (permalink / raw)
To: davem, mchan, eddie.wai, joe, mingo, peterz, netdev, linux-kernel; +Cc: paulmck
The "rcu_dereference()" calls are used directly in conditions.
Since their return values are never dereferenced it is recommended to use
"rcu_access_pointer()" instead of "rcu_dereference()".
Therefore, this patch makes the replacements.
The following Coccinelle semantic patch was used:
@@
@@
(
if(
(<+...
- rcu_dereference
+ rcu_access_pointer
(...)
...+>)) {...}
|
while(
(<+...
- rcu_dereference
+ rcu_access_pointer
(...)
...+>)) {...}
)
Signed-off-by: Andreea-Cristina Bernat <bernat.ada@gmail.com>
---
v2: Modified subject line from
"rcu: Replace rcu_dereference() with rcu_access_pointer()"
to
"cnic: Replace rcu_dereference() with rcu_access_pointer()",
noted by David Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/cnic.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/net/ethernet/broadcom/cnic.c b/drivers/net/ethernet/broadcom/cnic.c
index 8244e2b..dad9cb7 100644
--- a/drivers/net/ethernet/broadcom/cnic.c
+++ b/drivers/net/ethernet/broadcom/cnic.c
@@ -381,7 +381,7 @@ static int cnic_iscsi_nl_msg_recv(struct cnic_dev *dev, u32 msg_type,
break;
rcu_read_lock();
- if (!rcu_dereference(cp->ulp_ops[CNIC_ULP_L4])) {
+ if (!rcu_access_pointer(cp->ulp_ops[CNIC_ULP_L4])) {
rc = -ENODEV;
rcu_read_unlock();
break;
@@ -525,7 +525,7 @@ int cnic_unregister_driver(int ulp_type)
list_for_each_entry(dev, &cnic_dev_list, list) {
struct cnic_local *cp = dev->cnic_priv;
- if (rcu_dereference(cp->ulp_ops[ulp_type])) {
+ if (rcu_access_pointer(cp->ulp_ops[ulp_type])) {
pr_err("%s: Type %d still has devices registered\n",
__func__, ulp_type);
read_unlock(&cnic_dev_lock);
@@ -573,7 +573,7 @@ static int cnic_register_device(struct cnic_dev *dev, int ulp_type,
mutex_unlock(&cnic_lock);
return -EAGAIN;
}
- if (rcu_dereference(cp->ulp_ops[ulp_type])) {
+ if (rcu_access_pointer(cp->ulp_ops[ulp_type])) {
pr_err("%s: Type %d has already been registered to this device\n",
__func__, ulp_type);
mutex_unlock(&cnic_lock);
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] cnic: Replace rcu_dereference() with rcu_access_pointer()
2014-08-17 10:12 [PATCH v2] cnic: Replace rcu_dereference() with rcu_access_pointer() Andreea-Cristina Bernat
@ 2014-08-18 23:13 ` Michael Chan
2014-08-22 19:27 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: Michael Chan @ 2014-08-18 23:13 UTC (permalink / raw)
To: Andreea-Cristina Bernat
Cc: davem, eddie.wai, joe, mingo, peterz, netdev, linux-kernel, paulmck
On Sun, 2014-08-17 at 13:12 +0300, Andreea-Cristina Bernat wrote:
> The "rcu_dereference()" calls are used directly in conditions.
> Since their return values are never dereferenced it is recommended to use
> "rcu_access_pointer()" instead of "rcu_dereference()".
> Therefore, this patch makes the replacements.
>
> The following Coccinelle semantic patch was used:
> @@
> @@
>
> (
> if(
> (<+...
> - rcu_dereference
> + rcu_access_pointer
> (...)
> ...+>)) {...}
> |
> while(
> (<+...
> - rcu_dereference
> + rcu_access_pointer
> (...)
> ...+>)) {...}
> )
>
> Signed-off-by: Andreea-Cristina Bernat <bernat.ada@gmail.com>
Acked-by: Michael Chan <mchan@broadcom.com>
> ---
> v2: Modified subject line from
> "rcu: Replace rcu_dereference() with rcu_access_pointer()"
> to
> "cnic: Replace rcu_dereference() with rcu_access_pointer()",
> noted by David Miller <davem@davemloft.net>
>
> drivers/net/ethernet/broadcom/cnic.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/ethernet/broadcom/cnic.c b/drivers/net/ethernet/broadcom/cnic.c
> index 8244e2b..dad9cb7 100644
> --- a/drivers/net/ethernet/broadcom/cnic.c
> +++ b/drivers/net/ethernet/broadcom/cnic.c
> @@ -381,7 +381,7 @@ static int cnic_iscsi_nl_msg_recv(struct cnic_dev *dev, u32 msg_type,
> break;
>
> rcu_read_lock();
> - if (!rcu_dereference(cp->ulp_ops[CNIC_ULP_L4])) {
> + if (!rcu_access_pointer(cp->ulp_ops[CNIC_ULP_L4])) {
> rc = -ENODEV;
> rcu_read_unlock();
> break;
> @@ -525,7 +525,7 @@ int cnic_unregister_driver(int ulp_type)
> list_for_each_entry(dev, &cnic_dev_list, list) {
> struct cnic_local *cp = dev->cnic_priv;
>
> - if (rcu_dereference(cp->ulp_ops[ulp_type])) {
> + if (rcu_access_pointer(cp->ulp_ops[ulp_type])) {
> pr_err("%s: Type %d still has devices registered\n",
> __func__, ulp_type);
> read_unlock(&cnic_dev_lock);
> @@ -573,7 +573,7 @@ static int cnic_register_device(struct cnic_dev *dev, int ulp_type,
> mutex_unlock(&cnic_lock);
> return -EAGAIN;
> }
> - if (rcu_dereference(cp->ulp_ops[ulp_type])) {
> + if (rcu_access_pointer(cp->ulp_ops[ulp_type])) {
> pr_err("%s: Type %d has already been registered to this device\n",
> __func__, ulp_type);
> mutex_unlock(&cnic_lock);
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] cnic: Replace rcu_dereference() with rcu_access_pointer()
2014-08-17 10:12 [PATCH v2] cnic: Replace rcu_dereference() with rcu_access_pointer() Andreea-Cristina Bernat
2014-08-18 23:13 ` Michael Chan
@ 2014-08-22 19:27 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2014-08-22 19:27 UTC (permalink / raw)
To: bernat.ada
Cc: mchan, eddie.wai, joe, mingo, peterz, netdev, linux-kernel, paulmck
From: Andreea-Cristina Bernat <bernat.ada@gmail.com>
Date: Sun, 17 Aug 2014 13:12:09 +0300
> The "rcu_dereference()" calls are used directly in conditions.
> Since their return values are never dereferenced it is recommended to use
> "rcu_access_pointer()" instead of "rcu_dereference()".
> Therefore, this patch makes the replacements.
...
> Signed-off-by: Andreea-Cristina Bernat <bernat.ada@gmail.com>
Applied.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-08-22 19:27 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-17 10:12 [PATCH v2] cnic: Replace rcu_dereference() with rcu_access_pointer() Andreea-Cristina Bernat
2014-08-18 23:13 ` Michael Chan
2014-08-22 19:27 ` David Miller
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.