[RFC][PATCH] identify rcu-protected ptr

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please consider an addition to the patch to identify
raid rcu-protected pointer occurences.  All of the
work in the raid submittals which is correct is that
of Paul McKenney and any errors and oversights are
mine.

Also, please consider the effective addition of an
rcu_dereference() in bpq_get_ax25_dev() in recognition 
of each use being nested in an rcu critical section.
Thank you.

 md/md.c                 |    2 +-
 net/hamradio/bpqether.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

-------------------------------------------------

diff -urpNa -X dontdiff a/linux-2.6.14-rc3/drivers/md/md.c b/linux-2.6.14-rc3/drivers/md/md.c
--- a/linux-2.6.14-rc3/drivers/md/md.c	2005-09-30 14:17:35.000000000 -0700
+++ b/linux-2.6.14-rc3/drivers/md/md.c	2005-10-05 22:30:31.000000000 -0700
@@ -1145,7 +1145,7 @@ static int bind_rdev_to_array(mdk_rdev_t
 	}
 			
 	list_add(&rdev->same_set, &mddev->disks);
-	rdev->mddev = mddev;
+	rcu_assign_pointer(rdev->mddev, mddev);
 	printk(KERN_INFO "md: bind<%s>\n", bdevname(rdev->bdev,b));
 	return 0;
 }
diff -urpNa -X dontdiff a/linux-2.6.14-rc3/drivers/net/hamradio/bpqether.c b/linux-2.6.14-rc3/drivers/net/hamradio/bpqether.c
--- a/linux-2.6.14-rc3/drivers/net/hamradio/bpqether.c	2005-09-30 14:17:35.000000000 -0700
+++ b/linux-2.6.14-rc3/drivers/net/hamradio/bpqether.c	2005-10-05 22:32:53.000000000 -0700
@@ -144,7 +144,7 @@ static inline struct net_device *bpq_get
 {
 	struct bpqdev *bpq;
 
-	list_for_each_entry(bpq, &bpq_devices, bpq_list) {
+	list_for_each_entry_rcu(bpq, &bpq_devices, bpq_list) {
 		if (bpq->ethdev == dev)
 			return bpq->axdev;
 	}
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux