[PATCH 2/2] docs: make isa-debugcon example more useful / directly applicable

Laszlo Ersek lersek at redhat.com
Thu May 18 16:04:40 UTC 2023


On 5/18/23 16:34, Andrea Bolognani wrote:
> On Thu, May 18, 2023 at 02:59:39PM +0200, Laszlo Ersek wrote:
>> The type='pty' attribute in the <serial> element causes a Pseudo TTY to be
>> allocated on the host side via "/dev/ptmx", which is meant to be
>> interacted with via "virsh console" or similar.
>>
>> That's not how a firmware log is typically viewed or saved. Replace
>> type='pty' with type='file', and also provide an example <source> element
>> (with the pathname of the logfile), similarly to how the <serial> example
>> just above provides a <source> element too.
>>
>> Cc: "Daniel P. Berrangé" <berrange at redhat.com>
>> Cc: Andrea Bolognani <abologna at redhat.com>
>> Updates: 654968381df0256c047d2ecd4542ccc90dc57ad0
>> Signed-off-by: Laszlo Ersek <lersek at redhat.com>
>> ---
>>
>> Notes:
>>     context:-U6
>>
>>  docs/formatdomain.rst | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> I see you've already pushed the patches

I for sure have not, I don't have the right (that I know of)!

Hm... we have Jano to blame ^W^W to thank to, for the push :)

> but, just for completeness' sake:
> 
> Reviewed-by: Andrea Bolognani <abologna at redhat.com>

Thank you both!
Laszlo



More information about the libvir-list mailing list