From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: "Rafael J. Wysocki" Subject: Re: [patch 1/1] [PATCH] include storage keys in hibernation image. Date: Tue, 9 Aug 2011 21:56:12 +0200 References: <20110608074523.211912903@de.ibm.com> <201107290001.14465.rjw@sisk.pl> <20110809174524.45cd496d@mschwide> In-Reply-To: <20110809174524.45cd496d@mschwide> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201108092156.12451.rjw@sisk.pl> Sender: linux-kernel-owner@vger.kernel.org List-Archive: List-Post: To: Martin Schwidefsky Cc: linux-pm@lists.linux-foundation.org, linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org, Pavel Machek , Jiri Slaby List-ID: On Tuesday, August 09, 2011, Martin Schwidefsky wrote: > On Fri, 29 Jul 2011 00:01:14 +0200 > "Rafael J. Wysocki" wrote: > > > Sorry for the extreme delay. > > No big deal, I have been busy with other things anyway. > > > Having reconsidered things I think the code under the #ifdef above > > should really go to arch/s390. > > Ok, that is reasonable. > > > Now, for the purpose of exporting the headers I'd introduce > > CONFIG_ARCH_SAVE_PAGE_KEYS and make S390 do > > > > select ARCH_SAVE_PAGE_KEYS if HIBERNATION > > > > and I'd put a #ifdef depending on that into include/linux/suspend.h. > > > > Apart from this, I have only one complaint, which is that the kerneldoc > > comments should follow the standard (the other comments in snapshot.c don't, > > but that's a matter for a separate patch). > > Sounds good. I will come up with new patches for this and resend them > for review. Might be one or two weeks though, currently conferencing in > Orlando.. OK, thanks!