[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 4/4] hw/xen: Have Xen machines select 9pfs
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v2 4/4] hw/xen: Have Xen machines select 9pfs |
Date: |
Mon, 1 Feb 2021 12:03:51 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.6.0 |
On 2/1/21 11:23 AM, Paolo Bonzini wrote:
> On 01/02/21 10:18, Philippe Mathieu-Daudé wrote:
>> FYI using 'imply FSDEV_9P' instead I get:
>>
>> /usr/bin/ld: libcommon.fa.p/hw_xen_xen-legacy-backend.c.o: in function
>> `xen_be_register_common':
>> hw/xen/xen-legacy-backend.c:754: undefined reference to `xen_9pfs_ops'
>
> Ok, so then we have the case of a file (hw/xen/xen-legacy-backend.c)
> brought in by CONFIG_XEN. In that case this patch is incorrect...
>
>> The function is:
>>
>> void xen_be_register_common(void)
>> {
>> xen_set_dynamic_sysbus();
>>
>> xen_be_register("console", &xen_console_ops);
>> xen_be_register("vkbd", &xen_kbdmouse_ops);
>> #ifdef CONFIG_VIRTFS
>> xen_be_register("9pfs", &xen_9pfs_ops);
>> #endif
>> #ifdef CONFIG_USB_LIBUSB
>> xen_be_register("qusb", &xen_usb_ops);
>> #endif
>> }
>>
>> The object is compiled using:
>>
>> -- >8 --
>> -#ifdef CONFIG_VIRTFS
>> +#ifdef CONFIG_FSDEV_9P
>> xen_be_register("9pfs", &xen_9pfs_ops);
>> #endif
>> ---
>
> ... and this is the best fix, together with:
>
> - a "#include CONFIG_DEVICES" at the top (to get CONFIG_FSDEV_9P)
>
> - moving xen-legacy-backend.c from softmmu_ss to specific_ss (to get
> CONFIG_DEVICES)
>
> - changing "select" to "imply" in accel/Kconfig (otherwise the patch has
> no effect)
OK.
>
> But really, doing nothing and just dropping this patch is perfectly fine.
Yes, I'll respin what I have so far and continue when I find the
time and motivation another week-end.