Subject: Re: [PATCH v4 0/3] powerpc/pseries: fix issues in suspend/resume code

From: Tyrel Datwyler <[email-masked]>
Date: Tue, 25 Feb 2014 20:06:45 -0800

On 02/23/2014 05:53 PM, Benjamin Herrenschmidt wrote:
> On Wed, 2014-02-19 at 12:56 -0800, Tyrel Datwyler wrote:
>> This patchset fixes a couple of issues encountered in the suspend/resume code
>> base. First when using the kernel device tree update code update-nodes is
>> unnecessarily called more than once. Second the cpu cache lists are not
>> updated after a suspend/resume which under certain conditions may cause a
>> panic. Finally, since the cache list fix utilzes in kernel device tree update
>> code a means for telling drmgr that updating the device tree from userspace
>> is unnecessary.
> 
> This series breaks the !SMP build.
> 
> Cheers,
> Ben.

The second patch in this series created the build breakage. I've replied
to that patch with a new version that fixes the break.

-Tyrel

> 
>> Changes from v3:
>> - Updated patch descriptions to better reflect the behavior/interface changes
>>   and why they are acceptable per Ben's concerns.
>>
>> Changes from v2:
>> - Moved dynamic configuration update code into pseries specific routine
>>   per Nathan's suggestion.
>>
>> Changes from v1:
>> - Fixed several commit message typos
>> - Fixed authorship of first two patches
>>
>> Haren Myneni (2):
>>   powerpc/pseries: Device tree should only be updated once after
>>     suspend/migrate
>>   powerpc/pseries: Update dynamic cache nodes for suspend/resume
>>     operation
>>
>> Tyrel Datwyler (1):
>>   powerpc/pseries: Report in kernel device tree update to drmgr
>>
>>  arch/powerpc/include/asm/rtas.h           |  1 +
>>  arch/powerpc/platforms/pseries/mobility.c | 26 +++++++-----------
>>  arch/powerpc/platforms/pseries/suspend.c  | 44 ++++++++++++++++++++++++++++++-
>>  3 files changed, 54 insertions(+), 17 deletions(-)
>>
> 
>