<p dir="ltr"><br>
On 24 Feb 2016 10:10, "Thomas Huth" <<a href="mailto:thuth@redhat.com">thuth@redhat.com</a>> wrote:<br>
><br>
> On 24.02.2016 09:44, Dinar Valeev wrote:<br>
> > On Wed, Feb 24, 2016 at 9:34 AM, Alexey Kardashevskiy <<a href="mailto:aik@ozlabs.ru">aik@ozlabs.ru</a>> wrote:<br>
> >> On 02/24/2016 06:45 PM, Thomas Huth wrote:<br>
> >>> On 23.02.2016 16:15, Dinar Valeev wrote:<br>
> >>>> On Mon, Feb 15, 2016 at 7:09 PM, Thomas Huth <<a href="mailto:thuth@redhat.com">thuth@redhat.com</a>> wrote:<br>
> >>>>> On 10.02.2016 13:19, Thomas Huth wrote:<br>
> >>>>>> On 10.02.2016 01:33, Alexey Kardashevskiy wrote:<br>
> >>>>>>> On 02/09/2016 11:54 PM, Dinar Valeev wrote:<br>
> >>>>>>>> On Mon, Feb 8, 2016 at 6:50 PM, Thomas Huth <<a href="mailto:thuth@redhat.com">thuth@redhat.com</a>> wrote:<br>
> >>>>>>>>> On 08.02.2016 15:10, Dinar Valeev wrote:<br>
> >>>>>>>>>><br>
> >>>>>>>>>> I have long standing issue with SLOF. During start SLOF prints cp<br>
> >>>>>>>>>> codes with backspace:<br>
> >>>>>>>>>><br>
> >>>>>>>>>> <a href="https://github.com/aik/SLOF/blob/master/lib/libbootmsg/bootmsg_lvl.S#L61-L64">https://github.com/aik/SLOF/blob/master/lib/libbootmsg/bootmsg_lvl.S#L61-L64</a><br>
> >>>>>>>>>><br>
> >>>>>>>>>> We start a VM and record log, this is fine. But then when we look<br>
> >>>>>>>>>> at<br>
> >>>>>>>>>> the log through WebUI, because of backspace, Mozilla thinks this is<br>
> >>>>>>>>>> a<br>
> >>>>>>>>>> binary file.<br>
> >>><br>
> >>> ...<br>
> >>>>><br>
> >>>>> Just a completely different idea, but would the following patch also fix<br>
> >>>>> your issue?<br>
> >>>>><br>
> >>>>> diff --git a/lib/libbootmsg/bootmsg_lvl.S b/lib/libbootmsg/bootmsg_lvl.S<br>
> >>>>> index 2e4c135..14ce4bf 100644<br>
> >>>>> --- a/lib/libbootmsg/bootmsg_lvl.S<br>
> >>>>> +++ b/lib/libbootmsg/bootmsg_lvl.S<br>
> >>>>> @@ -58,10 +58,8 @@ ENTRY(bootmsg_cp)<br>
> >>>>>          bl      io_putchar      // print character<br>
> >>>>>          mr      r3, r9<br>
> >>>>>          bl      io_printhex16   // print checkpoint ID<br>
> >>>>> -       .rept   5<br>
> >>>>> -       li      r3,'\b'<br>
> >>>>> -       bl      io_putchar      // print backspaces<br>
> >>>>> -       .endr<br>
> >>>>> +       li      r3,'\r'<br>
> >>>>> +       bl      io_putchar      // go back<br>
> >>>>>          mtlr    r11<br>
> >>>>>          blr<br>
> >>>>><br>
> >>>>> Replacing 5 '\b's with one '\r' should IMHO be ok here since the<br>
> >>>>> checkpoints should always be printed at the beginning of a line...<br>
> >>>><br>
> >>>> I gave it a try.. It works flawlessly...<br>
> >>>><br>
> >>>> Could you commit it?<br>
> >>><br>
> >>><br>
> >>> That's a question for Alexey, I guess ... Alexey, what do you think, is<br>
> >>> that change ok?<br>
> > I mean should I send a patch or you could do it?<br>
><br>
> I just prepared and sent a patch.<br>
Thanks<br>
><br>
>  Thomas<br>
><br>
</p>