Re: [PATCH v4 3/3] vl: Abort if multiple machines are registered as default

2020-02-08 Thread David Gibson
On Fri, Feb 07, 2020 at 05:19:48PM +0100, Philippe Mathieu-Daudé wrote: > It would be confusing to have multiple default machines. > Abort if this ever occurs. > > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: David Gibson > --- > v2: Use assert() instead of human friendly message

Re: [PATCH v4 3/3] vl: Abort if multiple machines are registered as default

2020-02-07 Thread Laurent Vivier
Le 07/02/2020 à 17:19, Philippe Mathieu-Daudé a écrit : > It would be confusing to have multiple default machines. > Abort if this ever occurs. > > Signed-off-by: Philippe Mathieu-Daudé > --- > v2: Use assert() instead of human friendly message (Marc-André) > v3: Move the check to find_machine()

Re: [PATCH v4 3/3] vl: Abort if multiple machines are registered as default

2020-02-07 Thread Marc-André Lureau
On Fri, Feb 7, 2020 at 5:21 PM Philippe Mathieu-Daudé wrote: > > It would be confusing to have multiple default machines. > Abort if this ever occurs. > > Signed-off-by: Philippe Mathieu-Daudé > --- > v2: Use assert() instead of human friendly message (Marc-André) > v3: Move the check to

[PATCH v4 3/3] vl: Abort if multiple machines are registered as default

2020-02-07 Thread Philippe Mathieu-Daudé
It would be confusing to have multiple default machines. Abort if this ever occurs. Signed-off-by: Philippe Mathieu-Daudé --- v2: Use assert() instead of human friendly message (Marc-André) v3: Move the check to find_machine() (Michael) Cc: Marc-André Lureau Cc: Laurent Vivier Cc: Michael S.