[PATCH 0/1] acpi: apei: Enable APEI multiple GHES source to share an single external IRQ

[PATCH 0/1] acpi: apei: Enable APEI multiple GHES source to share an single external IRQ

[Mark Salter]

Rafael,

What do you think of this one? I was submitted a long time ago but
there was never any real resolution to it.


Loc Ho (1):
  acpi: apei: Enable APEI multiple GHES source to share an single
    external IRQ

 drivers/acpi/apei/ghes.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

-- 
2.13.3

Re: [PATCH 0/1] acpi: apei: Enable APEI multiple GHES source to share an single external IRQ

[Rafael J. Wysocki]

On Wed, Jul 19, 2017 at 7:24 PM, Mark Salter <msalter@redhat.com> wrote:
> Rafael,
>
> What do you think of this one?

Please ask this question to Boris Petkov in the first place.

> I was submitted a long time ago but
> there was never any real resolution to it.
>
>
> Loc Ho (1):
>   acpi: apei: Enable APEI multiple GHES source to share an single
>     external IRQ
>

Thanks,
Rafael

Re: [PATCH 0/1] acpi: apei: Enable APEI multiple GHES source to share an single external IRQ

[Mark Salter]

On Wed, 2017-07-19 at 21:23 +0200, Rafael J. Wysocki wrote:
> On Wed, Jul 19, 2017 at 7:24 PM, Mark Salter <msalter@redhat.com> wrote:
> > Rafael,
> > 
> > What do you think of this one?
> 
> Please ask this question to Boris Petkov in the first place.

Yes, I should have included Boris. The last time this came up, Boris
said it was up to you. :)

http://lists.infradead.org/pipermail/linux-arm-kernel/2016-January/401772.html

Re: [PATCH 0/1] acpi: apei: Enable APEI multiple GHES source to share an single external IRQ

[Borislav Petkov]

On Wed, Jul 19, 2017 at 04:05:21PM -0400, Mark Salter wrote:
> Yes, I should have included Boris. The last time this came up, Boris
> said it was up to you. :)
> 
> http://lists.infradead.org/pipermail/linux-arm-kernel/2016-January/401772.html

/me goes back and reads the old emails.

So I had some issues with how acpi_gsi_to_irq() returns the irq number
but that required more involved simplification and the submitter was
unsurprisingly reluctant to do it.

That's why it was Rafael's call.

AFAIR.

Also, from looking at the patches, the SOB chain is wrong:

Signed-off-by: Tuan Phan <tphan@apm.com>
Signed-off-by: Loc Ho <lho@apm.com>

I don't know from that who's the author and who's the submitter.
SubmittingPatches has more info on the whole deal.

So please resubmit updated versions with issues addressed.

Thanks.

-- 
Regards/Gruss,
    Boris.

SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)
-- 

Re: [PATCH 0/1] acpi: apei: Enable APEI multiple GHES source to share an single external IRQ

[Loc Ho]

Hi Borislav/Mark,

> On Wed, Jul 19, 2017 at 04:05:21PM -0400, Mark Salter wrote:
> > Yes, I should have included Boris. The last time this came up, Boris
> > said it was up to you. :)
> >
> > http://lists.infradead.org/pipermail/linux-arm-kernel/2016-January/401772.html
>
> /me goes back and reads the old emails.
>
> So I had some issues with how acpi_gsi_to_irq() returns the irq number
> but that required more involved simplification and the submitter was
> unsurprisingly reluctant to do it.
>
> That's why it was Rafael's call.
>
> AFAIR.
>
> Also, from looking at the patches, the SOB chain is wrong:
>
> Signed-off-by: Tuan Phan <tphan@apm.com>
> Signed-off-by: Loc Ho <lho@apm.com>
>
> I don't know from that who's the author and who's the submitter.
> SubmittingPatches has more info on the whole deal.
>
> So please resubmit updated versions with issues addressed.

Tuan and I both worked on the APEI side for X-Gene and for this one,
we both sign off. If you required, I can re-post another version. Or
you can simple pull in v4 2/2 patch -
http://lists.infradead.org/pipermail/linux-arm-kernel/2016-January/400780.html

-Loc

Re: [PATCH 0/1] acpi: apei: Enable APEI multiple GHES source to share an single external IRQ

[Borislav Petkov]

On Thu, Jul 20, 2017 at 10:49:08AM -0700, Loc Ho wrote:
> Tuan and I both worked on the APEI side for X-Gene and for this one,
> we both sign off.

There's no such thing as "both sign off". See the document I mentioned
before.

> If you required, I can re-post another version.

Yes please.

-- 
Regards/Gruss,
    Boris.

SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)
--