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

Groups > linux.kernel > #1693909 > unrolled thread

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

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to share a single external IRQ Loc Ho <lho@apm.com> - 2017-07-21 20:30 +0200
    Re: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to  share a single external IRQ Borislav Petkov <bp@suse.de> - 2017-07-22 08:10 +0200
      Re: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to share a single external IRQ "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-07-23 00:10 +0200
        Re: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to share a single external IRQ "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2017-07-26 21:10 +0200
          Re: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to  share a single external IRQ Borislav Petkov <bp@suse.de> - 2017-07-27 08:00 +0200
            Re: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to  share a single external IRQ "Rafael J. Wysocki" <rafael@kernel.org> - 2017-07-27 16:40 +0200
              Re: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to  share a single external IRQ Borislav Petkov <bp@suse.de> - 2017-07-27 16:50 +0200
                Re: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to  share a single external IRQ "Rafael J. Wysocki" <rafael@kernel.org> - 2017-07-27 17:10 +0200
                  RE: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to  share a single external IRQ "Luck, Tony" <tony.luck@intel.com> - 2017-07-27 19:00 +0200
                    [PATCH] MAINTAINERS: Add Tony and Boris as ACPI/APEI reviewers Borislav Petkov <bp@suse.de> - 2017-07-28 09:40 +0200
                      RE: [PATCH] MAINTAINERS: Add Tony and Boris as ACPI/APEI reviewers "Luck, Tony" <tony.luck@intel.com> - 2017-07-28 19:10 +0200

#1693909 — [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to share a single external IRQ

X-Gene platforms describe multiple GHES error sources with the same hardware
error notification type (external interrupt) and interrupt number.
Change the GHES interrupt request to support sharing the same IRQ.

Co-authored-by: Tuan Phan <tphan@apm.com>
Signed-off-by: Loc Ho <lho@apm.com>
---
 drivers/acpi/apei/ghes.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/acpi/apei/ghes.c b/drivers/acpi/apei/ghes.c
index d661d45..eed09fc 100644
--- a/drivers/acpi/apei/ghes.c
+++ b/drivers/acpi/apei/ghes.c
@@ -1157,7 +1157,8 @@ static int ghes_probe(struct platform_device *ghes_dev)
 			       generic->header.source_id);
 			goto err_edac_unreg;
 		}
-		rc = request_irq(ghes->irq, ghes_irq_func, 0, "GHES IRQ", ghes);
+		rc = request_irq(ghes->irq, ghes_irq_func, IRQF_SHARED,
+				 "GHES IRQ", ghes);
 		if (rc) {
 			pr_err(GHES_PFX "Failed to register IRQ for generic hardware error source: %d\n",
 			       generic->header.source_id);
-- 
1.8.3.1

[toc] | [next] | [standalone]

#1694111 — Re: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to share a single external IRQ

On Fri, Jul 21, 2017 at 11:24:37AM -0700, Loc Ho wrote:
> X-Gene platforms describe multiple GHES error sources with the same hardware
> error notification type (external interrupt) and interrupt number.
> Change the GHES interrupt request to support sharing the same IRQ.
> 
> Co-authored-by: Tuan Phan <tphan@apm.com>
> Signed-off-by: Loc Ho <lho@apm.com>
> ---
>  drivers/acpi/apei/ghes.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/acpi/apei/ghes.c b/drivers/acpi/apei/ghes.c
> index d661d45..eed09fc 100644
> --- a/drivers/acpi/apei/ghes.c
> +++ b/drivers/acpi/apei/ghes.c
> @@ -1157,7 +1157,8 @@ static int ghes_probe(struct platform_device *ghes_dev)
>  			       generic->header.source_id);
>  			goto err_edac_unreg;
>  		}
> -		rc = request_irq(ghes->irq, ghes_irq_func, 0, "GHES IRQ", ghes);
> +		rc = request_irq(ghes->irq, ghes_irq_func, IRQF_SHARED,
> +				 "GHES IRQ", ghes);
>  		if (rc) {
>  			pr_err(GHES_PFX "Failed to register IRQ for generic hardware error source: %d\n",
>  			       generic->header.source_id);
> -- 

Acked-by: Borislav Petkov <bp@suse.de>

-- 
Regards/Gruss,
    Boris.

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

[toc] | [prev] | [next] | [standalone]

#1694208

On Saturday, July 22, 2017 07:59:54 AM Borislav Petkov wrote:
> On Fri, Jul 21, 2017 at 11:24:37AM -0700, Loc Ho wrote:
> > X-Gene platforms describe multiple GHES error sources with the same hardware
> > error notification type (external interrupt) and interrupt number.
> > Change the GHES interrupt request to support sharing the same IRQ.
> > 
> > Co-authored-by: Tuan Phan <tphan@apm.com>
> > Signed-off-by: Loc Ho <lho@apm.com>
> > ---
> >  drivers/acpi/apei/ghes.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/acpi/apei/ghes.c b/drivers/acpi/apei/ghes.c
> > index d661d45..eed09fc 100644
> > --- a/drivers/acpi/apei/ghes.c
> > +++ b/drivers/acpi/apei/ghes.c
> > @@ -1157,7 +1157,8 @@ static int ghes_probe(struct platform_device *ghes_dev)
> >  			       generic->header.source_id);
> >  			goto err_edac_unreg;
> >  		}
> > -		rc = request_irq(ghes->irq, ghes_irq_func, 0, "GHES IRQ", ghes);
> > +		rc = request_irq(ghes->irq, ghes_irq_func, IRQF_SHARED,
> > +				 "GHES IRQ", ghes);
> >  		if (rc) {
> >  			pr_err(GHES_PFX "Failed to register IRQ for generic hardware error source: %d\n",
> >  			       generic->header.source_id);
> 
> Acked-by: Borislav Petkov <bp@suse.de>

OK

Should I apply this or will Tony do that?

[toc] | [prev] | [next] | [standalone]

#1697491

On Sunday, July 23, 2017 12:01:10 AM Rafael J. Wysocki wrote:
> On Saturday, July 22, 2017 07:59:54 AM Borislav Petkov wrote:
> > On Fri, Jul 21, 2017 at 11:24:37AM -0700, Loc Ho wrote:
> > > X-Gene platforms describe multiple GHES error sources with the same hardware
> > > error notification type (external interrupt) and interrupt number.
> > > Change the GHES interrupt request to support sharing the same IRQ.
> > > 
> > > Co-authored-by: Tuan Phan <tphan@apm.com>
> > > Signed-off-by: Loc Ho <lho@apm.com>
> > > ---
> > >  drivers/acpi/apei/ghes.c | 3 ++-
> > >  1 file changed, 2 insertions(+), 1 deletion(-)
> > > 
> > > diff --git a/drivers/acpi/apei/ghes.c b/drivers/acpi/apei/ghes.c
> > > index d661d45..eed09fc 100644
> > > --- a/drivers/acpi/apei/ghes.c
> > > +++ b/drivers/acpi/apei/ghes.c
> > > @@ -1157,7 +1157,8 @@ static int ghes_probe(struct platform_device *ghes_dev)
> > >  			       generic->header.source_id);
> > >  			goto err_edac_unreg;
> > >  		}
> > > -		rc = request_irq(ghes->irq, ghes_irq_func, 0, "GHES IRQ", ghes);
> > > +		rc = request_irq(ghes->irq, ghes_irq_func, IRQF_SHARED,
> > > +				 "GHES IRQ", ghes);
> > >  		if (rc) {
> > >  			pr_err(GHES_PFX "Failed to register IRQ for generic hardware error source: %d\n",
> > >  			       generic->header.source_id);
> > 
> > Acked-by: Borislav Petkov <bp@suse.de>
> 
> OK
> 
> Should I apply this or will Tony do that?

So am I suppsed to drop this?

[toc] | [prev] | [next] | [standalone]

#1697733 — Re: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to share a single external IRQ

On Wed, Jul 26, 2017 at 08:57:03PM +0200, Rafael J. Wysocki wrote:
> > Should I apply this or will Tony do that?
> 
> So am I suppsed to drop this?

Well, as suggested in another mail, I think it'll be best if Tony and I
put ourselves as reviewers for the APEI crap and you still collect it.

Unless you want to give it to us and we can arrange some pull order to
send it to you.

What do you prefer?

-- 
Regards/Gruss,
    Boris.

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

[toc] | [prev] | [next] | [standalone]

#1698034 — Re: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to share a single external IRQ

On Thu, Jul 27, 2017 at 7:58 AM, Borislav Petkov <bp@suse.de> wrote:
> On Wed, Jul 26, 2017 at 08:57:03PM +0200, Rafael J. Wysocki wrote:
>> > Should I apply this or will Tony do that?
>>
>> So am I suppsed to drop this?
>
> Well, as suggested in another mail, I think it'll be best if Tony and I
> put ourselves as reviewers for the APEI crap and you still collect it.
>
> Unless you want to give it to us and we can arrange some pull order to
> send it to you.
>
> What do you prefer?

I can collect it, no problem.

So from now on I will pick up APEI things with ACKs from you or Tony.

Thanks,
Rafael

[toc] | [prev] | [next] | [standalone]

#1698048 — Re: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to share a single external IRQ

On Thu, Jul 27, 2017 at 04:34:59PM +0200, Rafael J. Wysocki wrote:
> I can collect it, no problem.
> 
> So from now on I will pick up APEI things with ACKs from you or Tony.

If you agree with that, that is. But we kinda do that already, more or
less. :-)

Then I guess we should add something like this so that get_maintainer
finds us:

diff --git a/MAINTAINERS b/MAINTAINERS
index f66488dfdbc9..985413fb5d83 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -310,6 +310,14 @@ F:	drivers/pci/*/*acpi*
 F:	drivers/pci/*/*/*acpi*
 F:	tools/power/acpi/
 
+ACPI APEI
+M:	"Rafael J. Wysocki" <rjw@rjwysocki.net>
+M:	Len Brown <lenb@kernel.org>
+L:	linux-acpi@vger.kernel.org
+R:	Tony Luck <tony.luck@intel.com>
+R:	Borislav Petkov <bp@alien8.de>
+F:	drivers/acpi/apei/
+
 ACPI COMPONENT ARCHITECTURE (ACPICA)
 M:	Robert Moore <robert.moore@intel.com>
 M:	Lv Zheng <lv.zheng@intel.com>

---

Provided Tony agrees though... I'd venture a guess and say that he
doesn't have a choice, woahahhahaha...

:-)))

-- 
Regards/Gruss,
    Boris.

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

[toc] | [prev] | [next] | [standalone]

#1698076 — Re: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to share a single external IRQ

On Thu, Jul 27, 2017 at 4:49 PM, Borislav Petkov <bp@suse.de> wrote:
> On Thu, Jul 27, 2017 at 04:34:59PM +0200, Rafael J. Wysocki wrote:
>> I can collect it, no problem.
>>
>> So from now on I will pick up APEI things with ACKs from you or Tony.
>
> If you agree with that, that is. But we kinda do that already, more or
> less. :-)
>
> Then I guess we should add something like this so that get_maintainer
> finds us:
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index f66488dfdbc9..985413fb5d83 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -310,6 +310,14 @@ F: drivers/pci/*/*acpi*
>  F:     drivers/pci/*/*/*acpi*
>  F:     tools/power/acpi/
>
> +ACPI APEI
> +M:     "Rafael J. Wysocki" <rjw@rjwysocki.net>
> +M:     Len Brown <lenb@kernel.org>
> +L:     linux-acpi@vger.kernel.org
> +R:     Tony Luck <tony.luck@intel.com>
> +R:     Borislav Petkov <bp@alien8.de>
> +F:     drivers/acpi/apei/
> +
>  ACPI COMPONENT ARCHITECTURE (ACPICA)
>  M:     Robert Moore <robert.moore@intel.com>
>  M:     Lv Zheng <lv.zheng@intel.com>
>
> ---
>
> Provided Tony agrees though... I'd venture a guess and say that he
> doesn't have a choice, woahahhahaha...
>
> :-)))

Well, I guess send this officially with a CC:Tony and see what he says. :-)

[toc] | [prev] | [next] | [standalone]

#1698189 — RE: [PATCH v6 1/1] acpi: apei: Enable APEI multiple GHES source to share a single external IRQ

>> Provided Tony agrees though... I'd venture a guess and say that he
>> doesn't have a choice, woahahhahaha...
>>
>> :-)))
>
> Well, I guess send this officially with a CC:Tony and see what he says. :-)

That's definitely part of my day job ... so yes, please add me as a reviewer.

-Tony

[toc] | [prev] | [next] | [standalone]

#1698565 — [PATCH] MAINTAINERS: Add Tony and Boris as ACPI/APEI reviewers

Since this piece of the ACPI pile is doing RAS, it is perhaps prudent if
we at least paid attention to it and the direction it takes. So add Tony
and me as reviewers.

Signed-off-by: Borislav Petkov <bp@suse.de>
---
 MAINTAINERS | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index f66488dfdbc9..985413fb5d83 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -310,6 +310,14 @@ F:	drivers/pci/*/*acpi*
 F:	drivers/pci/*/*/*acpi*
 F:	tools/power/acpi/
 
+ACPI APEI
+M:	"Rafael J. Wysocki" <rjw@rjwysocki.net>
+M:	Len Brown <lenb@kernel.org>
+L:	linux-acpi@vger.kernel.org
+R:	Tony Luck <tony.luck@intel.com>
+R:	Borislav Petkov <bp@alien8.de>
+F:	drivers/acpi/apei/
+
 ACPI COMPONENT ARCHITECTURE (ACPICA)
 M:	Robert Moore <robert.moore@intel.com>
 M:	Lv Zheng <lv.zheng@intel.com>
-- 
2.14.0.rc0

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

[toc] | [prev] | [next] | [standalone]

#1698962 — RE: [PATCH] MAINTAINERS: Add Tony and Boris as ACPI/APEI reviewers

> Since this piece of the ACPI pile is doing RAS, it is perhaps prudent if
> we at least paid attention to it and the direction it takes. So add Tony
> and me as reviewers.

Acked-by: Tony Luck <tony.luck@intel.com>
 

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web