On 30.01.23 09:17, Juergen Gross wrote: > The Xenstore related documentation is currently to be found in > docs/misc/9pfs.pandoc, instead of the related header file > xen/include/public/io/9pfs.h like for most other paravirtualized > device protocols. > > There is a comment in the header pointing at the document, but the > given file name is wrong. Additionally such headers are meant to be > copied into consuming projects (Linux kernel, qemu, etc.), so pointing > at a doc file in the Xen git repository isn't really helpful for the > consumers of the header. > > This situation is far from ideal, which is already being proved by the > fact that neither qemu nor the Linux kernel are implementing the > device attach/detach protocol correctly. Additionally the documented > Xenstore entries are not matching the reality, as the "tag" Xenstore > entry is on the frontend side, not on the backend one. > > Change that by moving the Xenstore related 9pfs documentation from > docs/misc/9pfs.pandoc into xen/include/public/io/9pfs.h while fixing > the wrong Xenstore entry detail. > > Signed-off-by: Juergen Gross > --- > V2: > - add reference to header in the pandoc document (Jan Beulich) Oh, I just found another bug in the documentation of the connection protocol while trying to fix the Linux frontend. I'll send V3 of the patch soon. Juergen