On 29 July 2017 at 14:50, Patrick Steinhardt <p...@pks.im> wrote: > On Fri, Jul 28, 2017 at 02:20:49PM -0300, Philippe Mathieu-Daudé wrote: >> This is likely to break on BSD, but now than patchew has a NetBSD job >> you can trigger a build RESENDing this patch.
> Thanks for the feedback! Is this really relevant in this context, > though? Both 9p-local.c and 9p-handle.c already include linux > headers, especially <linux/fs.h> is included without any ifdef > around. As such, I simply assumed that this code is being built > on Linux systems, only. Yes; hw/Makefile.objs only includes the 9pfs directory if CONFIG_VIRTFS is defined, and configure only sets that on Linux hosts which have libpcap and libattr available. thanks -- PMM