Re: [PATCH v3 00/13] virtio-fs: shared file system for virtual machines

From: Vivek Goyal
Date: Thu Aug 29 2019 - 10:31:19 EST


On Thu, Aug 29, 2019 at 03:41:26PM +0200, Miklos Szeredi wrote:
> On Thu, Aug 29, 2019 at 3:29 PM Vivek Goyal <vgoyal@xxxxxxxxxx> wrote:
>
> > #ifdef CONFIG_VIRTIO_FS
> > /** virtio-fs's physically contiguous buffer for in and out args */
> > void *argbuf;
> > #endif
> >
> > It should have worked. Not sure why it is not working.
>
> Needs to be changed to
>
> #if IS_ENABLED(CONFIG_VIRTIO_FS)
>
> Pushed out fixed version.

Cool. That works. Faced another compilation error with "make allmodconfig"
config file.

HDRINST usr/include/linux/virtio_fs.h
error: include/uapi/linux/virtio_fs.h: missing "WITH Linux-syscall-note" for SPDX-License-Identifier
make[1]: *** [scripts/Makefile.headersinst:66: usr/include/linux/virtio_fs.h] Error 1

Looks like include/uapi/linux/virtio_fs.h needs following.

-/* SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause */
+/* SPDX-License-Identifier: ((GPL-2.0 WITH Linux-syscall-note) OR BSD-3-Clause) */

Vivek