16 Dec
2016
16 Dec
'16
7:38 p.m.
On Fri, Dec 16, 2016 at 07:25:16PM +0000, Chris Wilson wrote:
+static void __exit primes_exit(void) +{
- const struct primes *p;
- mutex_lock(&lock);
- p = rcu_dereference_protected(primes, lockdep_is_held(&lock));
- if (p != &small_primes) {
kfree_rcu((struct primes *)p, rcu);rcu_assign_pointer(p, &small_primes);
Too much sparse appleasing, too little thinking. It's only the module shutdown path, but we might as well be correct: s/p/primes here. -Chris
--
Chris Wilson, Intel Open Source Technology Centre