linux-kernel - Re: [v5 12/14] iio: imu: add BNO055 serdev driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAHp75VeEzuQS5rOK7t5Lyq1wQKVQLNoU_W1K1M20Jx=a5U96TQ@mail.gmail.com>
Date:   Tue, 3 May 2022 16:04:06 +0200
From:   Andy Shevchenko <andy.shevchenko@...il.com>
To:     Andrea Merello <andrea.merello@...il.com>
Cc:     kernel test robot <lkp@...el.com>,
        Jonathan Cameron <jic23@...nel.org>,
        Mauro Carvalho Chehab <mchehab+huawei@...nel.org>,
        linux-iio <linux-iio@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        devicetree <devicetree@...r.kernel.org>, kbuild-all@...ts.01.org,
        Lars-Peter Clausen <lars@...afoo.de>,
        Rob Herring <robh+dt@...nel.org>,
        Matt Ranostay <matt.ranostay@...sulko.com>,
        Alexandru Ardelean <ardeleanalex@...il.com>,
        jmondi <jacopo@...ndi.org>,
        Andrea Merello <andrea.merello@....it>
Subject: Re: [v5 12/14] iio: imu: add BNO055 serdev driver

On Tue, May 3, 2022 at 3:30 PM Andrea Merello <andrea.merello@...il.com> wrote:
>
> Il giorno mar 3 mag 2022 alle ore 09:48 Andrea Merello
> <andrea.merello@...il.com> ha scritto:
>
> [...]
>
> > > You need to add a C-file with the only line
> > >
> > > #include <..._trace.h>
> > >
> > > And drop that include from the _core.c.
> >
> > Hum, I'm a bit confused here: the bno055_ser_core.c file explicitly
> > looks for that tracepoints (e.g. it calls trace_send_chunks() and
> > friends); dropping the include prevents build here because there would
> > be no definition for those tracepoints.
> >
> > There is already a C file bno055_ser_trace.c that just contains the
> > said include and it defines CREATE_TRACE_POINTS; I see other drivers
> > like dwc3 do the same..
>
> Oops.. it turned out that I just had this almost-empty C file as
> untracked in my git tree, and it ended up not being included in
> patches also. Being it laying in my src tree caused the build to
> succeed.
>
> I have been misled by the other problem I (still) have (below); I was
> focused on the wrong thing, sorry.

So, there are two reports:
1) missed C file;
2) possible missed prototype.

To solve 1) you need to add the C file to the patch.
To solve 2) you need either declare it static or put it into the
header file (I haven't checked deeply which one is your case).

> > But my problem is that I cannot reproduce the issue found by the bot:
> > the compiler that is downloaded by the script doesn't run on my build
> > box because it wants a newer libc (I was hoping that those compilers
> > were statically linked, but they aren't), while any other attempt I
> > did with other older compilers resulted in either successful build or
> > failed with other weird, apparently unrelated, errors about relocation
> > issues (of course I tried with the arch and config used by the build
> > bot).

You may use compilers from kernel.org that don't require any libc at
all (only good for kernel compilation).

> > Is there any build farm publicly available or something like that?

Not of my knowledge.


-- 
With Best Regards,
Andy Shevchenko