Re: [PATCH v5 2/3] mmc: host: sdhci: Implement the request_atomic() API

From: Baolin Wang
Date: Mon Apr 13 2020 - 02:53:02 EST


Hi,

On Mon, Apr 13, 2020 at 1:42 PM kbuild test robot <lkp@xxxxxxxxx> wrote:
>
> Hi Baolin,
>
> I love your patch! Yet something to improve:
>
> [auto build test ERROR on linus/master]
> [also build test ERROR on v5.7-rc1 next-20200412]
> [cannot apply to arm-soc/for-next]
> [if your patch is applied to the wrong git tree, please drop us a note to help
> improve the system. BTW, we also suggest to use '--base' option to specify the
> base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
>
> url: https://github.com/0day-ci/linux/commits/Baolin-Wang/Introduce-the-request_atomic-for-the-host/20200413-105346
> base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 8f3d9f354286745c751374f5f1fcafee6b3f3136
> config: x86_64-randconfig-s2-20200413 (attached as .config)
> compiler: gcc-5 (Ubuntu 5.5.0-12ubuntu1) 5.5.0 20171010
> reproduce:
> # save the attached .config to linux build tree
> make ARCH=x86_64
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kbuild test robot <lkp@xxxxxxxxx>

This patch set is based on Adrian's patch set. Thanks.
https://www.spinics.net/lists/linux-mmc/msg58529.html

>
> All error/warnings (new ones prefixed by >>):
>
> In file included from include/linux/build_bug.h:5:0,
> from include/linux/bitfield.h:10,
> from drivers/mmc/host/sdhci.c:12:
> drivers/mmc/host/sdhci.c: In function 'sdhci_request_atomic':
> drivers/mmc/host/sdhci.c:2076:6: error: implicit declaration of function 'sdhci_present_error' [-Werror=implicit-function-declaration]
> if (sdhci_present_error(host, mrq->cmd, true)) {
> ^
> include/linux/compiler.h:58:52: note: in definition of macro '__trace_if_var'
> #define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
> ^
> >> drivers/mmc/host/sdhci.c:2076:2: note: in expansion of macro 'if'
> if (sdhci_present_error(host, mrq->cmd, true)) {
> ^
> drivers/mmc/host/sdhci.c:2081:8: error: implicit declaration of function 'sdhci_manual_cmd23' [-Werror=implicit-function-declaration]
> cmd = sdhci_manual_cmd23(host, mrq) ? mrq->sbc : mrq->cmd;
> ^
> drivers/mmc/host/sdhci.c:2090:2: error: invalid use of void expression
> if (!sdhci_send_command(host, cmd))
> ^
> drivers/mmc/host/sdhci.c:2090:2: error: invalid use of void expression
> In file included from include/linux/build_bug.h:5:0,
> from include/linux/bitfield.h:10,
> from drivers/mmc/host/sdhci.c:12:
> >> include/linux/compiler.h:61:16: error: invalid use of void expression
> static struct ftrace_branch_data \
> ^
> include/linux/compiler.h:58:69: note: in expansion of macro '__trace_if_value'
> #define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
> ^
> include/linux/compiler.h:56:28: note: in expansion of macro '__trace_if_var'
> #define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
> ^
> drivers/mmc/host/sdhci.c:2090:2: note: in expansion of macro 'if'
> if (!sdhci_send_command(host, cmd))
> ^
> cc1: some warnings being treated as errors
>
> vim +/if +2076 drivers/mmc/host/sdhci.c
>
> 2066
> 2067 int sdhci_request_atomic(struct mmc_host *mmc, struct mmc_request *mrq)
> 2068 {
> 2069 struct sdhci_host *host = mmc_priv(mmc);
> 2070 struct mmc_command *cmd;
> 2071 unsigned long flags;
> 2072 int ret = 0;
> 2073
> 2074 spin_lock_irqsave(&host->lock, flags);
> 2075
> > 2076 if (sdhci_present_error(host, mrq->cmd, true)) {
> 2077 sdhci_finish_mrq(host, mrq);
> 2078 goto out_finish;
> 2079 }
> 2080
> 2081 cmd = sdhci_manual_cmd23(host, mrq) ? mrq->sbc : mrq->cmd;
> 2082
> 2083 /*
> 2084 * The HSQ may send a command in interrupt context without polling
> 2085 * the busy signaling, which means we should return BUSY if controller
> 2086 * has not released inhibit bits to allow HSQ trying to send request
> 2087 * again in non-atomic context. So we should not finish this request
> 2088 * here.
> 2089 */
> 2090 if (!sdhci_send_command(host, cmd))
> 2091 ret = -EBUSY;
> 2092 else
> 2093 sdhci_led_activate(host);
> 2094
> 2095 out_finish:
> 2096 spin_unlock_irqrestore(&host->lock, flags);
> 2097 return ret;
> 2098 }
> 2099 EXPORT_SYMBOL_GPL(sdhci_request_atomic);
> 2100
>
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx



--
Baolin Wang