From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755384AbbA2Pkd (ORCPT ); Thu, 29 Jan 2015 10:40:33 -0500 Received: from outbound-smtp06.blacknight.com ([81.17.249.39]:47649 "EHLO outbound-smtp06.blacknight.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754226AbbA2Pk3 (ORCPT ); Thu, 29 Jan 2015 10:40:29 -0500 Message-ID: <54CA546B.3040704@nexus-software.ie> Date: Thu, 29 Jan 2015 15:40:27 +0000 From: "Bryan O'Donoghue" User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: "Ong, Boon Leong" CC: Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , "x86@kernel.org" , "dvhart@infradead.org" , "linux-kernel@vger.kernel.org" , Andy Shevchenko , Ingo Molnar Subject: Re: [PATCH 1/2] x86: Add Isolated Memory Regions for Quark X1000 References: <1422470186-7860-1-git-send-email-pure.logic@nexus-software.ie> <1422470186-7860-2-git-send-email-pure.logic@nexus-software.ie> <20150129074411.GA4817@gmail.com> <54CA0782.8000807@nexus-software.ie> <54CA502D.9030307@nexus-software.ie> In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 29/01/15 15:32, Ong, Boon Leong wrote: > > >> -----Original Message----- >> From: Bryan O'Donoghue [mailto:pure.logic@nexus-software.ie] >> Sent: Thursday, January 29, 2015 11:22 PM >> To: Ong, Boon Leong >> Cc: Thomas Gleixner; Ingo Molnar; H. Peter Anvin; x86@kernel.org; >> dvhart@infradead.org; linux-kernel@vger.kernel.org; Andy Shevchenko; Ingo >> Molnar >> Subject: Re: [PATCH 1/2] x86: Add Isolated Memory Regions for Quark X1000 >> >> On 29/01/15 13:47, Ong, Boon Leong wrote: >>> Bryan, >>> >>> Once you have the next revision ready, I would like to test it on my end across >> both Galileo Gen v1 & v2. >>> >>> Cheers, >>> BL >> >> Andy/BL - thanks for taking the time to test. >> >> I may end up dropping the imr_del_range() tests based on index as a result of >> changing the external interface as suggested by Ingo. > It would be nice to have two variants (1) index based & (2) address based. Understood. The direction from Ingo was to have address based external interface imr_del_range() and support an index based internal imr_clear() - internally. So - in order to get test coverage - I'll move the self-test code back into the main IMR code Not as pretty that way - but better coverage :)