On 13/11/2015 12:49, Peter Maydell wrote: > On 12 November 2015 at 09:48, Paolo Bonzini <[email protected]> wrote: >> >> >> On 11/11/2015 20:09, Eduardo Habkost wrote: >>> All DisplayType values are just UI options that don't affect any >>> hardware emulation code, except for DT_NOGRAPHIC. Replace >>> DT_NOGRAPHIC with DT_NONE plus a new MachineState.nographic >>> field, so hardware emulation code don't need to use the >>> display_type variable. >>> >>> Cc: Michael Walle <[email protected]> >>> Cc: Blue Swirl <[email protected]> >>> Cc: Mark Cave-Ayland <[email protected]> >>> Signed-off-by: Eduardo Habkost <[email protected]> >> >> Can you add a QOM property too, so that "-machine graphics=yes|no" can >> be used? > > We already have both '-nographic' and '-display none'. > I think adding yet another way to turn off graphics which isn't > the same as either of our existing command line options would > worsen this confusion...
I proposed the property exactly so that -nographic becomes the same as "-display none -machine graphics=no -serial mon:stdio". Eduardo's patches achieve that at thecode level, but not at the command line level. Paolo
