From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934363AbbIVTlj (ORCPT ); Tue, 22 Sep 2015 15:41:39 -0400 Received: from foss.arm.com ([217.140.101.70]:60137 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934124AbbIVTlg (ORCPT ); Tue, 22 Sep 2015 15:41:36 -0400 Date: Tue, 22 Sep 2015 20:41:34 +0100 From: Marc Zyngier To: Will Deacon Cc: Robert Richter , Thomas Gleixner , Jason Cooper , Tirumalesh Chalamarla , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Robert Richter , Catalin Marinas Subject: Re: [PATCH v5 0/6] irqchip, gicv3: Updates and Cavium ThunderX errata workarounds Message-ID: <20150922204134.7491db93@arm.com> In-Reply-To: <20150922182726.GN7356@arm.com> References: <1442869119-1814-1-git-send-email-rric@kernel.org> <20150922175701.65ee4437@arm.com> <20150922190932.4391652a@arm.com> <20150922182726.GN7356@arm.com> Organization: ARM Ltd X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.25; arm-unknown-linux-gnueabihf) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 22 Sep 2015 19:27:26 +0100 Will Deacon wrote: > On Tue, Sep 22, 2015 at 07:09:32PM +0100, Marc Zyngier wrote: > > On Tue, 22 Sep 2015 17:57:01 +0100 > > Marc Zyngier wrote: > > > > [Duh. Now with Will and Catalin on CC] > > > > > On Mon, 21 Sep 2015 22:58:33 +0200 > > > Robert Richter wrote: > > > > > > > From: Robert Richter > > > > > > > > This patch series adds gicv3 updates and workarounds for HW errata in > > > > Cavium's ThunderX GICV3. > > > > > > > > The patches has been rebased onto 4.3-rc1. Note that there are two > > > > important fixes. See below for all changes. > > > > > > > > The first one is an unchanged resubmission of a patch from a gicv3 > > > > series I sent a while ago. > > > > > > > > The next patches implement the workarounds for ThunderX's gicv3. Patch > > > > #2 implements the cpu workaround for gicv3 on ThunderX. Patch #3 is a > > > > prerequisit for patch #5. Patch #4 adds generic code to parse the hw > > > > revision provided by an IIDR. This patch is used for the implementa- > > > > tion of the actual gicv3-its workaround in #5. Patch #6 updates to the > > > > new jump label API. > > > > > > > > All current review comments addressed so far with v5 > > > > > > Catalin, Will: assuming you don't have any objection to this series, > > > how do you want to deal with patch 2? > > What are the actual dependencies here? AFAICT, the series is addressing > multiple errata, so would it be possible to make the arm64 bits somewhat > independent from the gic parts? Patch 2 could be split into an arm64-specific part and a gic part, with a bit of #ifdef-ery in gicv3_enable_quirks(). > Also, I assume this is targetting 4.4? Up to you, really. It is not a regression, but it would still be nice to have 4.3 working reliably on this HW. Thanks, M. -- Jazz is not dead. It just smells funny.