On Tue, Jun 29, 2010 at 04:55,  <[email protected]> wrote:
> blackfin: restore L1 base address and length to 0 after free
>
> Modified: trunk/arch/blackfin/include/asm/mmu_context.h (8951 => 8952)
>
>  static inline void free_l1stack(void)
>  {
>       nr_l1stack_tasks--;
> -     if (nr_l1stack_tasks == 0)

not specific to your change, but this l1stack code looks highly racey.
 shouldnt nr_l1stack-tasks be an atomic_t or something ?  and there's
no locking around the l1_stack_{base,len} ?
-mike
_______________________________________________
Linux-kernel-commits mailing list
[email protected]
https://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits

Reply via email to