Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1496468

Re: [PATCH] ARCv2: intc: untangle SMP, MCIP and IDU

Show all headers | View raw

Hi Vineet,

On Wed, 2016-10-05 at 13:39 -0700, Vineet Gupta wrote:
> The IDU intc is technically part of MCIP (Multi-core IP) hence
> historically was only available in a SMP hardware build (and thus only
> in a SMP kernel build). Now that hardware restriction has been lifted,
> so a UP kernel needs to support it.
> 
> This requires breaking mcip.c into parts which are strictly SMP
> (inter-core interrupts) and IDU which in reality is just another
> intc and thus has no bearing on SMP.
> 
> This change allows IDU in UP builds and with a suitable device tree, we
> can have the cascaded intc system
> 
>     ARCv2 core intc <---> ARCv2 IDU intc <---> periperals
> 
> Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
> ---

[snip]
 
> +struct mcip_bcr {
> +#ifdef CONFIG_CPU_BIG_ENDIAN
> +		unsigned int pad3:8,
> +			     idu:1, llm:1, num_cores:6,
> +			     iocoh:1,  gfrc:1, dbg:1, pad2:1,
> +			     msg:1, sem:1, ipi:1, pad:1,
> +			     ver:8;
> +#else
> +		unsigned int ver:8,
> +			     pad:1, ipi:1, sem:1, msg:1,
> +			     pad2:1, dbg:1, gfrc:1, iocoh:1,
> +			     num_cores:6, llm:1, idu:1,
> +			     pad3:8;
> +#endif
> +};

IMHO we should stop using this kind of constructions because they
are ugly and what's more important not portable.

Even though we have it now working for both LE and BE configurations
it won't work for 64-bit cores. We'll need to add ifdeffed 32-bit paddings
then which will make that construction even more ugly.

Probably that's not the right patch to address my complaint but just
to reiterate this topic once again and think about clean-up series on
that regard :)

-Alexey

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] ARCv2: intc: untangle SMP, MCIP and IDU Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2016-10-05 22:50 +0200
  Re: [PATCH] ARCv2: intc: untangle SMP, MCIP and IDU Alexey Brodkin <Alexey.Brodkin@synopsys.com> - 2016-10-06 11:20 +0200
    Re: [PATCH] ARCv2: intc: untangle SMP, MCIP and IDU Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2016-10-06 19:20 +0200
      Re: [PATCH] ARCv2: intc: untangle SMP, MCIP and IDU Alexey Brodkin <Alexey.Brodkin@synopsys.com> - 2016-10-07 19:40 +0200
        Re: [PATCH] ARCv2: intc: untangle SMP, MCIP and IDU Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2016-10-07 23:40 +0200

csiph-web