On Tue, Jan 21, 2020 at 4:19 PM Marek Marczykowski-Górecki <marma...@invisiblethingslab.com> wrote: > > On Mon, Jan 20, 2020 at 02:36:08PM -0500, Jason Andryuk wrote: > > On Tue, Jan 14, 2020 at 9:41 PM Marek Marczykowski-Górecki > > <marma...@invisiblethingslab.com> wrote: > > > > > > This allows using arguments with spaces, like -append, without > > > nominating any special "separator" character. > > > > > > Signed-off-by: Marek Marczykowski-Górecki > > > <marma...@invisiblethingslab.com> > > > --- > > > Changes in v3: > > > - previous version of this patch "libxl: use \x1b to separate qemu > > > arguments for linux stubdomain" used specific non-printable > > > separator, but it was rejected as xenstore doesn't cope well with > > > non-printable chars > > > --- > > > > The code looks good. > > > > Reviewed-by: Jason Andryuk <jandr...@gmail.com> > > > > One thought I have is dmargs is a string for mini-os and a directory > > for linux stubdom. It's toolstack managed, so it's not a problem. > > But would a different xenstore node be less surprising to humans? > > dmargs_list?
dmargs_list works. dmargv to mimic argv? That might be too subtle. I'm not asking for the change. I just wanted to bring it up for discussion. Regards, Jason _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
