Re: [PATCH 2/2] chardev: Allow setting file chardev input file on the command line

2023-04-14 Thread Marc-André Lureau
On Thu, Apr 13, 2023 at 7:07 PM Peter Maydell wrote: > Our 'file' chardev backend supports both "output from this chardev > is written to a file" and "input from this chardev should be read > from a file" (except on Windows). However, you can only set up > the input file if you're using the QMP i

Re: [PATCH 2/2] chardev: Allow setting file chardev input file on the command line

2023-04-14 Thread Thomas Huth
On 13/04/2023 17.07, Peter Maydell wrote: Our 'file' chardev backend supports both "output from this chardev is written to a file" and "input from this chardev should be read from a file" (except on Windows). However, you can only set up the input file if you're using the QMP interface -- there i

Re: [PATCH 2/2] chardev: Allow setting file chardev input file on the command line

2023-04-14 Thread Peter Maydell
On Fri, 14 Apr 2023 at 15:03, Philippe Mathieu-Daudé wrote: > > On 13/4/23 17:07, Peter Maydell wrote: > > --- a/qemu-options.hx > > +++ b/qemu-options.hx > > > > -``-chardev file,id=id,path=path`` > > +``-chardev file,id=id,path=path[,input-path=input-path]`` > > Log all traffic received fr

Re: [PATCH 2/2] chardev: Allow setting file chardev input file on the command line

2023-04-14 Thread Philippe Mathieu-Daudé
On 13/4/23 17:07, Peter Maydell wrote: Our 'file' chardev backend supports both "output from this chardev is written to a file" and "input from this chardev should be read from a file" (except on Windows). However, you can only set up the input file if you're using the QMP interface -- there is n

[PATCH 2/2] chardev: Allow setting file chardev input file on the command line

2023-04-13 Thread Peter Maydell
Our 'file' chardev backend supports both "output from this chardev is written to a file" and "input from this chardev should be read from a file" (except on Windows). However, you can only set up the input file if you're using the QMP interface -- there is no command line syntax to do it. Add comm