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 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?

thanks,

greg k-h
_______________________________________________
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