Re: [RFC][PATCH 2/2] Convert several sysdev users to using struct syscore_ops

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Friday, March 11, 2011, Greg KH wrote:
> On Fri, Mar 11, 2011 at 09:29:24PM +0100, Rafael J. Wysocki wrote:
> > I thought about two different possible ways forward:
> > 
> > (1) Push [1/2] and the patches converting things that x86 depends on first,
> >     followed perhaps by a patch introducing something like
> >     CONFIG_ARCH_NO_SYSDEV_OPS that would simply disable
> >     sysdev_{suspend|resume|shutdown}() (x86 would set it).  The other arches
> >     might then be converted over time.
> > 
> > (2) Prepare patches converting everything that can be converted in the tree
> >     and push them all in one shot.
> > 
> > The advantage of (1) is that we can start making changes RSN and the
> > advantage of (2) seems to be that we may avoid some potential suspend/resume
> > ordering issues on non-x86 architectures that may arise in principle if some
> > subsystems are converted to using struct syscore_ops while the others are
> > not (syscore_suspend() is executed after sysdev_suspend(), so if we move
> > something from the latter to the former, it may end up being executed after
> > things that it was executed before previously).
> > 
> > Please let me know what your opinion is.
> 
> Hm, I would prefer (1) as that lets us get this moving sooner, and "flag
> days" are never good to have.  If there are problems that arise because
> of it, as you have noted, it will be simple just to convert the parts
> that were using the "old" methods to the new ones to fix the issue,
> right?

Yes, I agree.

Thanks,
Rafael
_______________________________________________
linux-pm mailing list
linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linux-foundation.org/mailman/listinfo/linux-pm


[Index of Archives]     [Linux ACPI]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [CPU Freq]     [Kernel Newbies]     [Fedora Kernel]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux