[SLOF] [PATCH 2/3] virtio-serial: Rework shutdown sequence

[Segher Boessenkool]

Hi!

On Fri, Mar 13, 2020 at 11:31:46AM +1100, Alexey Kardashevskiy wrote:
> >>> It now also calls  virtio-serial-shutdown  if it never was initialised?
> >>
> >> No. This also removes "['] shutdown add-quiesce-xt" so the only way to
> >> end up in "shutdown" is having open-count incremented == open virtio-serial.
> > 
> > Anything can call shutdown, the nice simple name encourages that even.
> > It's not a good idea to make this word more fragile, imho.
> 
> In practice, one would need to run a guest with virtio-serial but not
> make it default stdout to avoid having it initialized, then select
> virtio-serial device in the SLOF prompt and call its "shutdown" method.
> This nice simple name is not that easily accessible ;)

Well, that depends on the user ;-)

My point is that this was bracketed with  initialized? IF ... THEN
before, which was more robust.  That is all.

Thanks,


Segher