Hi All,
The builds of loongarch loongson3_defconfig have failed to build
next-20220906 with the error:
drivers/scsi/qla2xxx/qla_os.c: In function 'qla_trace_init':
drivers/scsi/qla2xxx/qla_os.c:2854:25: error: implicit declaration of function 'trace_array_get_by_name'; did you mean 'trace_array_set_clr_event'? [-Werror=implicit-function-declaration]
2854 | qla_trc_array = trace_array_get_by_name("qla2xxx");
| ^~~~~~~~~~~~~~~~~~~~~~~
| trace_array_set_clr_event
drivers/scsi/qla2xxx/qla_os.c: In function 'qla_trace_uninit':
drivers/scsi/qla2xxx/qla_os.c:2869:9: error: implicit declaration of function 'trace_array_put' [-Werror=implicit-function-declaration]
2869 | trace_array_put(qla_trc_array);
| ^~~~~~~~~~~~~~~
git bisect pointed to 8bfc149ba24c ("scsi: qla2xxx: Enhance driver tracing with separate tunable and more").
I will be happy to test any patch or provide any extra log if needed.
--
Regards
Sudip
Hi Sudip,
On Tue, 6 Sep 2022, 7:32am, Sudip Mukherjee (Codethink) wrote:
> External Email
>
> ----------------------------------------------------------------------
> Hi All,
>
> The builds of loongarch loongson3_defconfig have failed to build
> next-20220906 with the error:
>
> drivers/scsi/qla2xxx/qla_os.c: In function 'qla_trace_init':
> drivers/scsi/qla2xxx/qla_os.c:2854:25: error: implicit declaration of function 'trace_array_get_by_name'; did you mean 'trace_array_set_clr_event'? [-Werror=implicit-function-declaration]
> 2854 | qla_trc_array = trace_array_get_by_name("qla2xxx");
> | ^~~~~~~~~~~~~~~~~~~~~~~
> | trace_array_set_clr_event
>
> drivers/scsi/qla2xxx/qla_os.c: In function 'qla_trace_uninit':
> drivers/scsi/qla2xxx/qla_os.c:2869:9: error: implicit declaration of function 'trace_array_put' [-Werror=implicit-function-declaration]
> 2869 | trace_array_put(qla_trc_array);
> | ^~~~~~~~~~~~~~~
>
>
> git bisect pointed to 8bfc149ba24c ("scsi: qla2xxx: Enhance driver tracing with separate tunable and more").
>
> I will be happy to test any patch or provide any extra log if needed.
>
>
This looks like is happening due to CONFIG_TRACING not being enabled (same
as what kernel test bot reported). Could you try out the fix attached?
@Steven, please see the patch attached. Do you think the fix belongs
better in linux/trace.h ?
Regards,
-Arun
On Tue, 6 Sep 2022 09:05:38 -0700
Arun Easi <[email protected]> wrote:
> This looks like is happening due to CONFIG_TRACING not being enabled (same
> as what kernel test bot reported). Could you try out the fix attached?
>
> @Steven, please see the patch attached. Do you think the fix belongs
> better in linux/trace.h ?
Yes, of course. Feel free to send me a patch.
Thanks!
-- Steve
On Tue, Sep 6, 2022 at 5:05 PM Arun Easi <[email protected]> wrote:
>
> Hi Sudip,
>
> On Tue, 6 Sep 2022, 7:32am, Sudip Mukherjee (Codethink) wrote:
>
> > External Email
> >
> > ----------------------------------------------------------------------
> > Hi All,
> >
> > The builds of loongarch loongson3_defconfig have failed to build
> > next-20220906 with the error:
> >
> > drivers/scsi/qla2xxx/qla_os.c: In function 'qla_trace_init':
> > drivers/scsi/qla2xxx/qla_os.c:2854:25: error: implicit declaration of function 'trace_array_get_by_name'; did you mean 'trace_array_set_clr_event'? [-Werror=implicit-function-declaration]
> > 2854 | qla_trc_array = trace_array_get_by_name("qla2xxx");
> > | ^~~~~~~~~~~~~~~~~~~~~~~
> > | trace_array_set_clr_event
> >
> > drivers/scsi/qla2xxx/qla_os.c: In function 'qla_trace_uninit':
> > drivers/scsi/qla2xxx/qla_os.c:2869:9: error: implicit declaration of function 'trace_array_put' [-Werror=implicit-function-declaration]
> > 2869 | trace_array_put(qla_trc_array);
> > | ^~~~~~~~~~~~~~~
> >
> >
> > git bisect pointed to 8bfc149ba24c ("scsi: qla2xxx: Enhance driver tracing with separate tunable and more").
> >
> > I will be happy to test any patch or provide any extra log if needed.
> >
> >
>
> This looks like is happening due to CONFIG_TRACING not being enabled (same
> as what kernel test bot reported). Could you try out the fix attached?
Thanks, that fixed the build for me. The patch did not apply directly
on next-20220906, so I had to modify it a little bit.
Tested-by: Sudip Mukherjee <[email protected]>
--
Regards
Sudip
On Tue, 6 Sep 2022 21:57:10 +0100
Sudip Mukherjee <[email protected]> wrote:
> Thanks, that fixed the build for me. The patch did not apply directly
> on next-20220906, so I had to modify it a little bit.
>
> Tested-by: Sudip Mukherjee <[email protected]>
Well, the change needs to go into include/linux/trace.h though.
-- Steve
On Tue, 6 Sep 2022, 2:03pm, Steven Rostedt wrote:
> On Tue, 6 Sep 2022 21:57:10 +0100
> Sudip Mukherjee <[email protected]> wrote:
>
> > Thanks, that fixed the build for me. The patch did not apply directly
> > on next-20220906, so I had to modify it a little bit.
> >
> > Tested-by: Sudip Mukherjee <[email protected]>
>
> Well, the change needs to go into include/linux/trace.h though.
>
Steve, I was thinking both the fixes stay at least for a short
term the one in qla2xxx to avoid tree dependencies.
Since, the qla_def.h change goes as:
+#ifndef CONFIG_TRACING
+#ifndef trace_array_get_by_name
..it should co-exist when the "#define trace_array_get_by_name" gets
merged in include/linux/trace.h. BTW, I will send out the changes to
trace.h today.
Other alternatives/suggestions welcome.
Regards,
-Arun
On Tue, 6 Sep 2022 14:26:31 -0700
Arun Easi <[email protected]> wrote:
> Steve, I was thinking both the fixes stay at least for a short
> term the one in qla2xxx to avoid tree dependencies.
>
> Since, the qla_def.h change goes as:
>
> +#ifndef CONFIG_TRACING
> +#ifndef trace_array_get_by_name
>
> ..it should co-exist when the "#define trace_array_get_by_name" gets
> merged in include/linux/trace.h. BTW, I will send out the changes to
> trace.h today.
>
> Other alternatives/suggestions welcome.
I doubt I'll have anything that conflicts with an update to
include/linux/trace.h, as it is seldom modified.
Just sent a patch out that updates that file and I'll review it, and then
you can push it through your tree.
-- Steve
On Tue, 6 Sep 2022, 2:41pm, Steven Rostedt wrote:
> On Tue, 6 Sep 2022 14:26:31 -0700
> Arun Easi <[email protected]> wrote:
>
> > Steve, I was thinking both the fixes stay at least for a short
> > term the one in qla2xxx to avoid tree dependencies.
> >
> > Since, the qla_def.h change goes as:
> >
> > +#ifndef CONFIG_TRACING
> > +#ifndef trace_array_get_by_name
> >
> > ..it should co-exist when the "#define trace_array_get_by_name" gets
> > merged in include/linux/trace.h. BTW, I will send out the changes to
> > trace.h today.
> >
> > Other alternatives/suggestions welcome.
>
> I doubt I'll have anything that conflicts with an update to
> include/linux/trace.h, as it is seldom modified.
>
> Just sent a patch out that updates that file and I'll review it, and then
> you can push it through your tree.
>
Sounds good. Thanks Steve.
Regards,
-Arun