All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] unmodified-drivers: tolerate IRQF_DISABLED being undefined
@ 2015-06-01 13:50 Jan Beulich
  2015-06-01 13:56 ` Ian Campbell
  0 siblings, 1 reply; 6+ messages in thread
From: Jan Beulich @ 2015-06-01 13:50 UTC (permalink / raw)
  To: xen-devel

It's being removed in Linux 4.1.

Signed-off-by: Jan Beulich <jbeulich@suse.com>

--- a/unmodified_drivers/linux-2.6/platform-pci/evtchn.c
+++ b/unmodified_drivers/linux-2.6/platform-pci/evtchn.c
@@ -350,11 +350,13 @@ int xen_irq_init(struct pci_dev *pdev)
 #if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,22)
 			   SA_SHIRQ | SA_SAMPLE_RANDOM | SA_INTERRUPT,
 #else
-			   IRQF_SHARED |
 #ifdef IRQF_SAMPLE_RANDOM
 			   IRQF_SAMPLE_RANDOM |
 #endif
-			   IRQF_DISABLED,
+#ifdef IRQF_DISABLED
+			   IRQF_DISABLED |
+#endif
+			   IRQF_SHARED,
 #endif
 			   "xen-platform-pci", pdev);
 }

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] unmodified-drivers: tolerate IRQF_DISABLED being undefined
  2015-06-01 13:50 [PATCH] unmodified-drivers: tolerate IRQF_DISABLED being undefined Jan Beulich
@ 2015-06-01 13:56 ` Ian Campbell
  2015-06-01 14:18   ` Jan Beulich
  0 siblings, 1 reply; 6+ messages in thread
From: Ian Campbell @ 2015-06-01 13:56 UTC (permalink / raw)
  To: Jan Beulich; +Cc: xen-devel

On Mon, 2015-06-01 at 14:50 +0100, Jan Beulich wrote:
> It's being removed in Linux 4.1.
> 
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Not sure who should, but:

Acked-by: Ian Campbell <ian.campbell@citrix.com>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] unmodified-drivers: tolerate IRQF_DISABLED being undefined
  2015-06-01 13:56 ` Ian Campbell
@ 2015-06-01 14:18   ` Jan Beulich
  2015-07-21  6:52     ` Olaf Hering
  0 siblings, 1 reply; 6+ messages in thread
From: Jan Beulich @ 2015-06-01 14:18 UTC (permalink / raw)
  To: Ian Campbell; +Cc: xen-devel

>>> On 01.06.15 at 15:56, <ian.campbell@citrix.com> wrote:
> On Mon, 2015-06-01 at 14:50 +0100, Jan Beulich wrote:
>> It's being removed in Linux 4.1.
>> 
>> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> 
> Not sure who should, but:

I guess no-one really needs to for that old code. But thanks!

Jan

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] unmodified-drivers: tolerate IRQF_DISABLED being undefined
  2015-06-01 14:18   ` Jan Beulich
@ 2015-07-21  6:52     ` Olaf Hering
  2015-07-21  7:02       ` Jan Beulich
  0 siblings, 1 reply; 6+ messages in thread
From: Olaf Hering @ 2015-07-21  6:52 UTC (permalink / raw)
  To: Jan Beulich; +Cc: xen-devel, Ian Campbell

On Mon, Jun 01, Jan Beulich wrote:

> >>> On 01.06.15 at 15:56, <ian.campbell@citrix.com> wrote:
> > On Mon, 2015-06-01 at 14:50 +0100, Jan Beulich wrote:
> >> It's being removed in Linux 4.1.
> >> 
> >> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> > 
> > Not sure who should, but:
> 
> I guess no-one really needs to for that old code. But thanks!

Any chance to backport this to all 4.x branches?

Olaf

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] unmodified-drivers: tolerate IRQF_DISABLED being undefined
  2015-07-21  6:52     ` Olaf Hering
@ 2015-07-21  7:02       ` Jan Beulich
  2015-07-21  7:07         ` Olaf Hering
  0 siblings, 1 reply; 6+ messages in thread
From: Jan Beulich @ 2015-07-21  7:02 UTC (permalink / raw)
  To: Olaf Hering; +Cc: xen-devel

>>> On 21.07.15 at 08:52, <olaf@aepfle.de> wrote:
> On Mon, Jun 01, Jan Beulich wrote:
> 
>> >>> On 01.06.15 at 15:56, <ian.campbell@citrix.com> wrote:
>> > On Mon, 2015-06-01 at 14:50 +0100, Jan Beulich wrote:
>> >> It's being removed in Linux 4.1.
>> >> 
>> >> Signed-off-by: Jan Beulich <jbeulich@suse.com>
>> > 
>> > Not sure who should, but:
>> 
>> I guess no-one really needs to for that old code. But thanks!
> 
> Any chance to backport this to all 4.x branches?

Taking it to mean "all maintained", that's possible, but since I can't
immediately see the point I'd like to ask for some justification. Us -
SUSE - apparently being the only consumer of that code, and us
not using Xen 4.4 together with Linux 4.1, and our 4.5 trees
already having the backport, I'm having some difficulty
understanding the reason for the request.

Thanks, Jan

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] unmodified-drivers: tolerate IRQF_DISABLED being undefined
  2015-07-21  7:02       ` Jan Beulich
@ 2015-07-21  7:07         ` Olaf Hering
  0 siblings, 0 replies; 6+ messages in thread
From: Olaf Hering @ 2015-07-21  7:07 UTC (permalink / raw)
  To: Jan Beulich; +Cc: xen-devel

On Tue, Jul 21, Jan Beulich wrote:

> Taking it to mean "all maintained", that's possible, but since I can't
> immediately see the point I'd like to ask for some justification. Us -
> SUSE - apparently being the only consumer of that code, and us
> not using Xen 4.4 together with Linux 4.1, and our 4.5 trees
> already having the backport, I'm having some difficulty
> understanding the reason for the request.

I run compile tests of all xen-4.x branches against all SUSE releases to
spot new errors. But since all the required changes for gcc wont be
backported anyway I will just patch also this issues out of my copy.

Olaf

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2015-07-21  7:08 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-01 13:50 [PATCH] unmodified-drivers: tolerate IRQF_DISABLED being undefined Jan Beulich
2015-06-01 13:56 ` Ian Campbell
2015-06-01 14:18   ` Jan Beulich
2015-07-21  6:52     ` Olaf Hering
2015-07-21  7:02       ` Jan Beulich
2015-07-21  7:07         ` Olaf Hering

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.