Re: [PATCH 2/2] libxl: devd: Spawn QEMU for 9pfs

[Anthony PERARD <anthony.perard@xxxxxxxxx>]

On Tue, Jan 09, 2024 at 12:05:40PM -0500, Jason Andryuk wrote:
> @@ -3430,7 +3431,7 @@ void libxl__spawn_qdisk_backend(libxl__egc *egc, 
> libxl__dm_spawn_state *dmss)
>       * because we will call this from unprivileged driver domains,
>       * so save it in the current domain libxl private dir.
>       */
> -    dmss->spawn.pidpath = GCSPRINTF("libxl/%u/qdisk-backend-pid", domid);
> +    dmss->spawn.pidpath = GCSPRINTF("libxl/%u/qemu-xenpv-backend-pid", 
> domid);

This path is documented in "docs/misc/xenstore-paths.pandoc", I'm not
sure it's such a good idea to change it.

But maybe it's ok to change the path because xl devd is expected add
then remove this path? And that pid isn't going to be useful to anything
else?

I'd rather not change the path, but if you still think it's ok, we can
go for it. Thought?

Thanks,

-- 
Anthony PERARD