On Fri, Apr 19, 2013 at 10:09 PM, Russell King - ARM Linux <li...@arm.linux.org.uk> wrote: > On Sat, Apr 13, 2013 at 09:08:12PM +0800, Haojian Zhuang wrote: >> On Thu, Apr 11, 2013 at 11:39 AM, Neil Zhang <zhan...@marvell.com> wrote: >> > + /* >> > + * Synchronise with the boot thread. >> > + */ >> > + spin_lock(&boot_lock); >> > + spin_unlock(&boot_lock); >> Lock & unlock without protecting anything. If so, you can remove this. > > ... which means you don't understand what is going on in this code, > and probably didn't read the comment above this fragment. The above > is to synchronise with the code below - again, read the comment at > the spin unlock in this function, remembering that the above code > and the code below runs concurrently on two different CPUs: > Yes, I misunderstood it. Thanks for your correction.
Regards Haojian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/