* [PATCH] Drivers: hv: vmbus: include header for get_irq_regs()
@ 2018-08-30 7:55 Sebastian Andrzej Siewior
2018-09-10 14:06 ` Sebastian Andrzej Siewior
2018-09-15 23:26 ` Michael Kelley (EOSG)
0 siblings, 2 replies; 6+ messages in thread
From: Sebastian Andrzej Siewior @ 2018-08-30 7:55 UTC (permalink / raw)
To: K. Y. Srinivasan, Haiyang Zhang, Stephen Hemminger
Cc: devel, linux-kernel, Steven Rostedt, Bernhard Landauer, Ralf Ramsauer
On !RT the header file get_irq_regs() gets pulled in via other header files. On
RT it does not and the build fails:
drivers/hv/vmbus_drv.c:975 implicit declaration of function ‘get_irq_regs’ [-Werror=implicit-function-declaration]
drivers/hv/hv.c:115 implicit declaration of function ‘get_irq_regs’ [-Werror=implicit-function-declaration]
Add the header file for get_irq_regs() in a common header so it used by
vmbus_drv.c by hv.c for their get_irq_regs() usage.
Reported-by: Bernhard Landauer <oberon@manjaro.org>
Reported-by: Ralf Ramsauer <ralf.ramsauer@oth-regensburg.de>
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
---
drivers/hv/hyperv_vmbus.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/hv/hyperv_vmbus.h b/drivers/hv/hyperv_vmbus.h
index 72eaba3d50fc2..797f07918197c 100644
--- a/drivers/hv/hyperv_vmbus.h
+++ b/drivers/hv/hyperv_vmbus.h
@@ -31,6 +31,7 @@
#include <linux/atomic.h>
#include <linux/hyperv.h>
#include <linux/interrupt.h>
+#include <linux/irq.h>
#include "hv_trace.h"
--
2.18.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] Drivers: hv: vmbus: include header for get_irq_regs()
2018-08-30 7:55 [PATCH] Drivers: hv: vmbus: include header for get_irq_regs() Sebastian Andrzej Siewior
@ 2018-09-10 14:06 ` Sebastian Andrzej Siewior
2018-09-10 21:08 ` KY Srinivasan
2018-09-15 23:26 ` Michael Kelley (EOSG)
1 sibling, 1 reply; 6+ messages in thread
From: Sebastian Andrzej Siewior @ 2018-09-10 14:06 UTC (permalink / raw)
To: K. Y. Srinivasan, Haiyang Zhang, Stephen Hemminger
Cc: devel, linux-kernel, Steven Rostedt, Bernhard Landauer, Ralf Ramsauer
On 2018-08-30 09:55:03 [+0200], To K. Y. Srinivasan wrote:
> On !RT the header file get_irq_regs() gets pulled in via other header files. On
> RT it does not and the build fails:
>
> drivers/hv/vmbus_drv.c:975 implicit declaration of function ‘get_irq_regs’ [-Werror=implicit-function-declaration]
> drivers/hv/hv.c:115 implicit declaration of function ‘get_irq_regs’ [-Werror=implicit-function-declaration]
>
> Add the header file for get_irq_regs() in a common header so it used by
> vmbus_drv.c by hv.c for their get_irq_regs() usage.
>
> Reported-by: Bernhard Landauer <oberon@manjaro.org>
> Reported-by: Ralf Ramsauer <ralf.ramsauer@oth-regensburg.de>
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
ping
> ---
> drivers/hv/hyperv_vmbus.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/hv/hyperv_vmbus.h b/drivers/hv/hyperv_vmbus.h
> index 72eaba3d50fc2..797f07918197c 100644
> --- a/drivers/hv/hyperv_vmbus.h
> +++ b/drivers/hv/hyperv_vmbus.h
> @@ -31,6 +31,7 @@
> #include <linux/atomic.h>
> #include <linux/hyperv.h>
> #include <linux/interrupt.h>
> +#include <linux/irq.h>
>
> #include "hv_trace.h"
>
> --
> 2.18.0
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] Drivers: hv: vmbus: include header for get_irq_regs()
2018-09-10 14:06 ` Sebastian Andrzej Siewior
@ 2018-09-10 21:08 ` KY Srinivasan
2018-09-11 8:03 ` Sebastian Andrzej Siewior
0 siblings, 1 reply; 6+ messages in thread
From: KY Srinivasan @ 2018-09-10 21:08 UTC (permalink / raw)
To: Sebastian Andrzej Siewior, Haiyang Zhang, Stephen Hemminger
Cc: devel, linux-kernel, Steven Rostedt, Bernhard Landauer, Ralf Ramsauer
> -----Original Message-----
> From: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> Sent: Monday, September 10, 2018 7:07 AM
> To: KY Srinivasan <kys@microsoft.com>; Haiyang Zhang
> <haiyangz@microsoft.com>; Stephen Hemminger
> <sthemmin@microsoft.com>
> Cc: devel@linuxdriverproject.org; linux-kernel@vger.kernel.org; Steven
> Rostedt <rostedt@goodmis.org>; Bernhard Landauer
> <oberon@manjaro.org>; Ralf Ramsauer <ralf.ramsauer@oth-
> regensburg.de>
> Subject: Re: [PATCH] Drivers: hv: vmbus: include header for get_irq_regs()
>
> On 2018-08-30 09:55:03 [+0200], To K. Y. Srinivasan wrote:
> > On !RT the header file get_irq_regs() gets pulled in via other header files.
> On
> > RT it does not and the build fails:
> >
> > drivers/hv/vmbus_drv.c:975 implicit declaration of function
> ‘get_irq_regs’ [-Werror=implicit-function-declaration]
> > drivers/hv/hv.c:115 implicit declaration of function ‘get_irq_regs’ [-
> Werror=implicit-function-declaration]
> >
> > Add the header file for get_irq_regs() in a common header so it used by
> > vmbus_drv.c by hv.c for their get_irq_regs() usage.
> >
> > Reported-by: Bernhard Landauer <oberon@manjaro.org>
> > Reported-by: Ralf Ramsauer <ralf.ramsauer@oth-regensburg.de>
> > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
>
> ping
It is in my queue; will submit it soon.
K. Y
>
> > ---
> > drivers/hv/hyperv_vmbus.h | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/hv/hyperv_vmbus.h b/drivers/hv/hyperv_vmbus.h
> > index 72eaba3d50fc2..797f07918197c 100644
> > --- a/drivers/hv/hyperv_vmbus.h
> > +++ b/drivers/hv/hyperv_vmbus.h
> > @@ -31,6 +31,7 @@
> > #include <linux/atomic.h>
> > #include <linux/hyperv.h>
> > #include <linux/interrupt.h>
> > +#include <linux/irq.h>
> >
> > #include "hv_trace.h"
> >
> > --
> > 2.18.0
> >
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Drivers: hv: vmbus: include header for get_irq_regs()
2018-09-10 21:08 ` KY Srinivasan
@ 2018-09-11 8:03 ` Sebastian Andrzej Siewior
0 siblings, 0 replies; 6+ messages in thread
From: Sebastian Andrzej Siewior @ 2018-09-11 8:03 UTC (permalink / raw)
To: KY Srinivasan
Cc: Haiyang Zhang, Stephen Hemminger, devel, linux-kernel,
Steven Rostedt, Bernhard Landauer, Ralf Ramsauer
On 2018-09-10 21:08:38 [+0000], KY Srinivasan wrote:
> It is in my queue; will submit it soon.
thank you for letting me know.
> K. Y
Sebastian
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] Drivers: hv: vmbus: include header for get_irq_regs()
2018-08-30 7:55 [PATCH] Drivers: hv: vmbus: include header for get_irq_regs() Sebastian Andrzej Siewior
2018-09-10 14:06 ` Sebastian Andrzej Siewior
@ 2018-09-15 23:26 ` Michael Kelley (EOSG)
2018-09-17 7:54 ` Sebastian Andrzej Siewior
1 sibling, 1 reply; 6+ messages in thread
From: Michael Kelley (EOSG) @ 2018-09-15 23:26 UTC (permalink / raw)
To: Sebastian Andrzej Siewior, KY Srinivasan, Haiyang Zhang,
Stephen Hemminger
Cc: devel, linux-kernel, Steven Rostedt, Bernhard Landauer, Ralf Ramsauer
From Sebastian Andrzej Siewior Sent: Thursday, August 30, 2018 12:55 AM
>
> On !RT the header file get_irq_regs() gets pulled in via other header files. On
> RT it does not and the build fails:
>
> drivers/hv/vmbus_drv.c:975 implicit declaration of function ‘get_irq_regs’ [-
> Werror=implicit-function-declaration]
> drivers/hv/hv.c:115 implicit declaration of function ‘get_irq_regs’ [-Werror=implicit-
> function-declaration]
>
> Add the header file for get_irq_regs() in a common header so it used by
> vmbus_drv.c by hv.c for their get_irq_regs() usage.
>
get_irq_regs() is not used explicitly in either vmbus_drv.c or in hv.c. And I
couldn't make the line numbers in the errors above line up with anything
in the source code that might be implicitly using get_irq_regs(). Is it the
calls to add_interrupt_randomness()? Did you figure out
exactly what line of code is causing the compile error?
I'm wondering whether adding the #include of irq.h into hyperv_vmbus.h
is really the right solution. More correct might be to have the file where
get_irq_regs() is actually used to #include irq_regs.h.
Michael
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Drivers: hv: vmbus: include header for get_irq_regs()
2018-09-15 23:26 ` Michael Kelley (EOSG)
@ 2018-09-17 7:54 ` Sebastian Andrzej Siewior
0 siblings, 0 replies; 6+ messages in thread
From: Sebastian Andrzej Siewior @ 2018-09-17 7:54 UTC (permalink / raw)
To: Michael Kelley (EOSG)
Cc: KY Srinivasan, Haiyang Zhang, Stephen Hemminger, devel,
linux-kernel, Steven Rostedt, Bernhard Landauer, Ralf Ramsauer
On 2018-09-15 23:26:53 [+0000], Michael Kelley (EOSG) wrote:
> From Sebastian Andrzej Siewior Sent: Thursday, August 30, 2018 12:55 AM
> >
> > On !RT the header file get_irq_regs() gets pulled in via other header files. On
> > RT it does not and the build fails:
> >
> > drivers/hv/vmbus_drv.c:975 implicit declaration of function ‘get_irq_regs’ [-
> > Werror=implicit-function-declaration]
> > drivers/hv/hv.c:115 implicit declaration of function ‘get_irq_regs’ [-Werror=implicit-
> > function-declaration]
> >
> > Add the header file for get_irq_regs() in a common header so it used by
> > vmbus_drv.c by hv.c for their get_irq_regs() usage.
> >
>
> get_irq_regs() is not used explicitly in either vmbus_drv.c or in hv.c. And I
> couldn't make the line numbers in the errors above line up with anything
> in the source code that might be implicitly using get_irq_regs(). Is it the
> calls to add_interrupt_randomness()? Did you figure out
> exactly what line of code is causing the compile error?
The line mentioned in the patch, is correct. I just figured out that
function is added as part preempt-RT.
> I'm wondering whether adding the #include of irq.h into hyperv_vmbus.h
> is really the right solution. More correct might be to have the file where
> get_irq_regs() is actually used to #include irq_regs.h.
nah, leave it. I'm sorry, that I haven't seen it earlier. I will submit
a complete patch once I get there. It popped during a stable update so I
assumed it was something that came via stable but it actually was there
the whole time. Again, sorry for the noise.
> Michael
Sebastian
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-09-17 7:54 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-08-30 7:55 [PATCH] Drivers: hv: vmbus: include header for get_irq_regs() Sebastian Andrzej Siewior
2018-09-10 14:06 ` Sebastian Andrzej Siewior
2018-09-10 21:08 ` KY Srinivasan
2018-09-11 8:03 ` Sebastian Andrzej Siewior
2018-09-15 23:26 ` Michael Kelley (EOSG)
2018-09-17 7:54 ` Sebastian Andrzej Siewior
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).