From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Carstens Subject: Re: [PATCHv2 00/14] set_memory_* functions header refactor Date: Thu, 2 Mar 2017 08:34:26 +0100 Message-ID: <20170302073426.GA7265@osiris> References: <1488413706-9739-1-git-send-email-labbott@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1488413706-9739-1-git-send-email-labbott@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Laura Abbott Cc: Mark Rutland , David Airlie , Catalin Marinas , Will Deacon , linux-kernel@vger.kernel.org, Wim Van Sebroeck , "H. Peter Anvin" , linux-watchdog@vger.kernel.org, linux-arch@vger.kernel.org, linux-s390@vger.kernel.org, Jessica Yu , Daniel Borkmann , x86@kernel.org, Russell King , Alexander Shishkin , Ingo Molnar , Guenter Roeck , Kees Cook , Arnd Bergmann , Thomas Gleixner , linux-arm-kernel@lists.infradead.org, Takashi Iwai , Martin Schwidefsky , Andrew Morton , "David S. Miller" List-Id: linux-arch.vger.kernel.org On Wed, Mar 01, 2017 at 04:14:52PM -0800, Laura Abbott wrote: > Hi, > > This is v2 of my proposal to move set_memory_* function prototypes out of > cacheflush.h and into their own header file. This came out of a comment > Russell made while reviewing RODATA test cases > http://lists.infradead.org/pipermail/linux-arm-kernel/2017-January/480855.html > While the final result of that series was the rodata code was refactored into > its own header file, the set_memory_* APIs are still out of place. > > This version refactored the common set_memory_* functions into an asm-generic > header. s390x added some features so it can't just use the asm-generic header. > I debated how much more to try and shove into the asm-generic version (e.g. > stub prototypes for the ARM nommu case, set_kernel_text_*) but decided to stick > with just the basics for this version. > > I split out the cacheflush.h -> set_memory.h conversions into separate patches > to hopefully make merging easier. Worst case, the final patch to completely > separate the two can be delayed if there are more problems found. I'd like > for this to eventually go through the -mm tree so I'd like Acks where > appropriate. > > As always, feedback appreciated. For the s390 bits: Acked-by: Heiko Carstens From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:44503 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754424AbdCBJBe (ORCPT ); Thu, 2 Mar 2017 04:01:34 -0500 Received: from pps.filterd (m0098410.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.20/8.16.0.20) with SMTP id v227SwgN086734 for ; Thu, 2 Mar 2017 02:34:40 -0500 Received: from e06smtp05.uk.ibm.com (e06smtp05.uk.ibm.com [195.75.94.101]) by mx0a-001b2d01.pphosted.com with ESMTP id 28wxrc30ee-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Thu, 02 Mar 2017 02:34:39 -0500 Received: from localhost by e06smtp05.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Thu, 2 Mar 2017 07:34:36 -0000 Date: Thu, 2 Mar 2017 08:34:26 +0100 From: Heiko Carstens Subject: Re: [PATCHv2 00/14] set_memory_* functions header refactor References: <1488413706-9739-1-git-send-email-labbott@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1488413706-9739-1-git-send-email-labbott@redhat.com> Message-ID: <20170302073426.GA7265@osiris> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Laura Abbott Cc: Russell King , Catalin Marinas , Will Deacon , Martin Schwidefsky , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , Kees Cook , Mark Rutland , Arnd Bergmann , David Airlie , Alexander Shishkin , Wim Van Sebroeck , Guenter Roeck , linux-watchdog@vger.kernel.org, "David S. Miller" , Daniel Borkmann , Jessica Yu , Takashi Iwai , Andrew Morton , x86@kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org, linux-arch@vger.kernel.org Message-ID: <20170302073426.TgrSR44kG5fCjFpQuWy1KYB0HFme0-sbgQTA7UICpyY@z> On Wed, Mar 01, 2017 at 04:14:52PM -0800, Laura Abbott wrote: > Hi, > > This is v2 of my proposal to move set_memory_* function prototypes out of > cacheflush.h and into their own header file. This came out of a comment > Russell made while reviewing RODATA test cases > http://lists.infradead.org/pipermail/linux-arm-kernel/2017-January/480855.html > While the final result of that series was the rodata code was refactored into > its own header file, the set_memory_* APIs are still out of place. > > This version refactored the common set_memory_* functions into an asm-generic > header. s390x added some features so it can't just use the asm-generic header. > I debated how much more to try and shove into the asm-generic version (e.g. > stub prototypes for the ARM nommu case, set_kernel_text_*) but decided to stick > with just the basics for this version. > > I split out the cacheflush.h -> set_memory.h conversions into separate patches > to hopefully make merging easier. Worst case, the final patch to completely > separate the two can be delayed if there are more problems found. I'd like > for this to eventually go through the -mm tree so I'd like Acks where > appropriate. > > As always, feedback appreciated. For the s390 bits: Acked-by: Heiko Carstens