linux-kernel - Re: [PATCH v10 16/21] irqchip: Add GICv2 specific ACPI boot support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20150314114315.GC9696@e104818-lin.cambridge.arm.com>
Date:	Sat, 14 Mar 2015 11:43:15 +0000
From:	Catalin Marinas <catalin.marinas@....com>
To:	Grant Likely <grant.likely@...aro.org>
Cc:	Jason Cooper <jason@...edaemon.net>,
	Mark Rutland <mark.rutland@....com>,
	linaro-acpi <linaro-acpi@...ts.linaro.org>,
	Will Deacon <will.deacon@....com>,
	Hanjun Guo <guohanjun@...wei.com>,
	Lorenzo Pieralisi <Lorenzo.Pieralisi@....com>,
	Timur Tabi <timur@...eaurora.org>,
	ACPI Devel Mailing List <linux-acpi@...r.kernel.org>,
	Robert Richter <rric@...nel.org>,
	Arnd Bergmann <arnd@...db.de>,
	Marc Zyngier <marc.zyngier@....com>,
	Jon Masters <jcm@...hat.com>,
	Tomasz Nowicki <tomasz.nowicki@...aro.org>,
	Mark Brown <broonie@...nel.org>,
	Thomas Gleixner <tglx@...utronix.de>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	Graeme Gregory <graeme.gregory@...aro.org>,
	Ashwin Chaugule <ashwinc@...eaurora.org>,
	"Rafael J. Wysocki" <rjw@...ysocki.net>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Hanjun Guo <hanjun.guo@...aro.org>,
	Suravee Suthikulanit <suravee.suthikulpanit@....com>,
	Sudeep Holla <Sudeep.Holla@....com>,
	Olof Johansson <olof@...om.net>
Subject: Re: [PATCH v10 16/21] irqchip: Add GICv2 specific ACPI boot support

On Sat, Mar 14, 2015 at 08:47:49AM +0000, Grant Likely wrote:
> On Fri, Mar 13, 2015 at 5:15 PM, Jason Cooper <jason@...edaemon.net> wrote:
> > Hanjun, Catalin,
> >
> > On Thu, Mar 12, 2015 at 03:31:57PM +0800, Hanjun Guo wrote:
> >> On 2015/3/12 13:12, Jason Cooper wrote:
> >> > On Thu, Mar 12, 2015 at 09:46:39AM +0800, Hanjun Guo wrote:
> >> >> On 2015/3/12 7:11, Jason Cooper wrote:
> >> >>> Hey Grant,
> >> >>>
> >> >>> On Wed, Mar 11, 2015 at 06:04:50PM +0000, Grant Likely wrote:
> >> >>>> On 11 Mar 2015 12:42, "Hanjun Guo" <hanjun.guo@...aro.org> wrote:
> >> [...]
> >> >>>>> diff --git a/drivers/irqchip/irqchip.c b/drivers/irqchip/irqchip.c
> >> >>>>> index 0fe2f71..afd1af3 100644
> >> >>>>> --- a/drivers/irqchip/irqchip.c
> >> >>>>> +++ b/drivers/irqchip/irqchip.c
> >> >>>>> @@ -8,6 +8,7 @@
> >> >>>>>   * warranty of any kind, whether express or implied.
> >> >>>>>   */
> >> >>>>>
> >> >>>>> +#include <linux/acpi_irq.h>
> >> >>>>>  #include <linux/init.h>
> >> >>>>>  #include <linux/of_irq.h>
> >> >>>>>  #include <linux/irqchip.h>
> >> >>>>> @@ -26,4 +27,6 @@ extern struct of_device_id __irqchip_of_table[];
> >> >>>>>  void __init irqchip_init(void)
> >> >>>>>  {
> >> >>>>>         of_irq_init(__irqchip_of_table);
> >> >>>>> +
> >> >>>>> +       acpi_irq_init();
> >> >>>>>  }
> >> >>> Is this in line with Olof's idea that providing a dtb would override ACPI?
> >> >> Yes, it will. Since ACPI is default OFF (disabled), if a dtb provided, and no acpi=force
> >> >> passed in the early command line, dtb will be used as system configuration for
> >> >> boot (dtb is always the prior one for now) [1]. In acpi_gic_init() which called by
> >> >> acpi_irq_init(), it will return immediately if acpi disabled, so it will not parse
> >> >> any ACPI table for device configuration.
> >> > Ok, that matches my recollection.  Thanks for refreshing my memory.  I'll apply
> >> > this on a topic branch for irqchip/gic when I return from travel.  Most likely
> >> > Friday or over the weekend.
> >>
> >> Thank you very much! But this patch can't be applied without previous ones in this
> >> patch set, how about you ack this patch and Catalin takes it via ARM64 tree? I'm
> >> not sure for this, it depends on your decision.
> >
> > Is this a build dependency or a boot dependency?  I only received this patch in
> > the series and I apologize, I'm a bit swamped atm.  Catalin, would an immutable
> > irqchip/gic topic branch with this in it work for you?
> 
> Jason,
> 
> For a series like this I strongly recommend you provide an ack and let
> the whole series go in via a single branch. Trying to split it up only
> to reassemble it again creates more work for everyone. There is also
> very little likelyhood that this will create a complex conflict with
> your tree.

I would prefer this approach as well since we only enable ACPI on arm64
on patch 19. If we are to rework this, we probably end up with 3
branches: one for the base ACPI, another for irqchip and yet another to
enable ACPI on arm64 (that's unless we enable APCI on arm64 but without
any irqchip support which makes it pretty useless for testing).

-- 
Catalin
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/