linux-kernel - Re: submit.c:27:17: error: expected ')' before '__VA_OPT__' current->comm __VA_OPT__(,) __VA_ARGS_

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: <f889adb9-dc01-79e3-12fb-1e66485246df@nvidia.com>
Date:   Fri, 16 Jul 2021 11:46:18 +0100
From:   Jon Hunter <jonathanh@...dia.com>
To:     Naresh Kamboju <naresh.kamboju@...aro.org>,
        Linux-Next Mailing List <linux-next@...r.kernel.org>,
        open list <linux-kernel@...r.kernel.org>,
        <lkft-triage@...ts.linaro.org>,
        clang-built-linux <clang-built-linux@...glegroups.com>
CC:     Stephen Rothwell <sfr@...b.auug.org.au>,
        Mikko Perttunen <mperttunen@...dia.com>,
        Thierry Reding <treding@...dia.com>,
        linux-tegra <linux-tegra@...r.kernel.org>,
        Daniel Vetter <daniel@...ll.ch>,
        David Airlie <airlied@...ux.ie>,
        Nathan Chancellor <natechancellor@...il.com>
Subject: Re: submit.c:27:17: error: expected ')' before '__VA_OPT__'
 current->comm __VA_OPT__(,) __VA_ARGS__)


On 15/07/2021 19:36, Naresh Kamboju wrote:
> Regression detected on  Linux next tag 20210715 for arm64 due to the
> following patch with
>  - gcc-7 - FAILED
>  -  clang-10 - FAILED
>  -  clang-11- FAILED
> But PASS with gcc-11 and clang-12
> 
> drm/tegra: Implement job submission part of new UAPI
> Implement the job submission IOCTL with a minimum feature set.
> 
> Signed-off-by: Mikko Perttunen <mperttunen@...dia.com>
> 
> Build error:
> ------------
>   CC [M]  drivers/gpu/drm/tegra/submit.o
> In file included include/linux/device.h:15:0,
>                  include/linux/host1x.h:9,
>                  drivers/gpu/drm/tegra/submit.c:6:
> drivers/gpu/drm/tegra/submit.c: In function 'submit_copy_gather_data':
> drivers/gpu/drm/tegra/submit.c:27:17: error: expected ')' before '__VA_OPT__'
>    current->comm __VA_OPT__(,) __VA_ARGS__)
>                  ^
> Reported-by: Linux Kernel Functional Testing <lkft@...aro.org>
> 
> ref:
> https://gitlab.com/Linaro/lkft/mirrors/next/linux-next/-/jobs/1425953551#L197
> 
> https://ci.linaro.org/view/lkft/job/openembedded-lkft-linux-next/DISTRO=lkft,MACHINE=juno,label=docker-buster-lkft/1068/consoleText


Thanks, yes I am seeing the same. The following fixes this and I will
send a patch to get this corrected.

diff --git a/drivers/gpu/drm/tegra/submit.c b/drivers/gpu/drm/tegra/submit.c
index c53b7207c478..e49630089149 100644
--- a/drivers/gpu/drm/tegra/submit.c
+++ b/drivers/gpu/drm/tegra/submit.c
@@ -24,7 +24,7 @@
 #define SUBMIT_ERR(context, fmt, ...) \
        dev_err_ratelimited(context->client->base.dev, \
                "%s: job submission failed: " fmt "\n", \
-               current->comm __VA_OPT__(,) __VA_ARGS__)
+               current->comm, ##__VA_ARGS__)

 static struct tegra_drm_mapping *
 tegra_drm_mapping_get(struct tegra_drm_context *context, u32 id)


Cheers
Jon

-- 
nvpublic