Re: [RFC PATCH 0/5] Add smp booting support for Qualcomm ARMv8 SoCs

From: Lorenzo Pieralisi
Date: Fri Apr 10 2015 - 12:07:32 EST


On Fri, Apr 10, 2015 at 04:25:54PM +0100, Kumar Gala wrote:
>
> On Apr 10, 2015, at 6:03 AM, Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx> wrote:
>
> > On Thu, Apr 09, 2015 at 06:37:06PM +0100, Kumar Gala wrote:
> >> This patch set adds support for SMP boot on the MSM8x16 family of Qualcomm SoCs.
> >>
> >> To support SMP on the MSM8x16 SoCs we need to add ARMv8/64-bit SCM interfaces to
> >> setup the boot/release addresses for the secondary CPUs. In addition we need
> >> a uniquie set of cpu ops. I'm aware the desired methods for booting secondary
> >> CPUs is either via spintable or PSCI. However, these SoCs are shipping with a
> >> firmware that does not support those methods.
> >
> > Why ? Do not tell me you were not aware of those standard methods,
> > because I can't believe you.
> >
> > If there were additional features to add to spin-table and PSCI,
> > you were, you are and you will always be welcome to debate them.
> >
> > There is no justification for this patchset, honestly.
> >
> > Lorenzo
>
> The justification for this patchset is support for a hardware platform that exists in the world. The kernel usual is willing to accept such things as long as the code is reasonable.

You are telling me you add a linker section for cpu_ops just for
"a platform that exists", sorry I do not believe that (and that's *not*
reasonable).

I do not like this line of reasoning, at all, because you were aware of
PSCI and ignored it, deliberately. Start by pushing code for platforms
upstream that support PSCI, "they are coming up next" does not cut it.

I can see power management mach code coming next, just no way.

Implement PSCI and the kernel will be willing to accept it, as it stands
as far as I am concerned that's a NAK on the series.

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