On Wed, 19 Dec 2007, Daniel Walker wrote:
> > It looks like the swsusp_save() calls drain_all_pages() , which calls
> > on_each_cpu() .. On return on_each_cpu() unconditionally enables
> > interrupts so the rest of the resume process has interrupt enable
> > (which , it looks like, shouldn't happen) and then you get the lockdep()
> > warning due to the above..
> >
> > Not sure if this has been found already, or not?
Hmmm... It will unconditionally enable interrupts regardless how we call
this. We could explicity save and restore interrrupts in
swsusp_save() I guess. Why is swsusp_save() disabling interrupts?
> > Should drain_all_pages() really be drain_local_pages() ?
>
> It looks like it was drain_local_pages, but the following patch
>
> page-allocator-clean-up-pcp-draining-functions.patch
>
> Changes that in -mm .. I added Christoph Lameter to the CC since it's
> his patch ..
We could reexport drain_local_pages() again but then I do not understand
why we would only drain the pages of this processor and not of all other
processors as well. It seems that software suspend intend was to flush
them all right?
--
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]
