On Mon, 2014-01-20 at 16:39 +0800, Tiejun Chen wrote:
>       if (atomic_read(&tlb->mm->mm_users) < 2 ||
>           cpumask_equal(mm_cpumask(tlb->mm),
>                         cpumask_of(smp_processor_id()))) {
>               kmem_cache_free(hugepte_cache, hugepte);
> +        put_cpu_var(hugepd_freelist_cur);
>               return;
>       }
>  

Whitespace

> @@ -491,6 +492,7 @@ static void hugepd_free(struct mmu_gather *tlb, void 
> *hugepte)
>               call_rcu_sched(&(*batchp)->rcu, hugepd_free_rcu_callback);
>               *batchp = NULL;
>       }
> +     put_cpu_var(hugepd_freelist_cur);
>  }

A blank line before "put_cpu_var" would be nice.

-Scott


_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to