Re: [PATCH] menu_entry: Disable cursor during update_screen()

[Thomas Huth]

 Hi Andrei,

On 13.02.2016 08:41, Andrei Borzenkov wrote:
> 31.01.2016 18:02, Thomas Huth пишет:
>>
>> On 30.01.2016 08:33, Andrei Borzenkov wrote:
>>> 29.01.2016 17:26, Thomas Huth пишет:
>>>> When running grub in a VGA console of a KVM pseries guest on PowerPC,
>>>> you can see the cursor sweeping over the whole line when entering a
>>>> character in editor mode. This is visible because grub always refreshes
>>>> the whole line when entering a character in editor mode, and drawing
>>>> characters is quite a slow operation with the firmware used for the
>>>> powerpc pseries guests (SLOF).
>>>> To avoid this ugliness, the cursor should be disabled when refreshing
>>>> the screen contents during update_screen().
>>>
>>> While it looks OK - any reason it is not present in Fedora grub2 GIT?
>>
>> Sorry, I didn't quite got your question ... do you mean the problem can
>> not be seen with the version from Fedora? If so, how did you start QEMU
>> to run the guest?
> 
> I usually expect that patches from distributions initially appear in
> distribution packages, so I was surprised that I could not see it in
> Fedora package.

Ah, ok, simple answer: Although I'm working for Red Hat, I am not
involved in the Fedora project (yet). And the problem with the cursor
during update_screen() has been found while testing RHEL, not Fedora, so
discussing this issue directly upstream here seems to make more sense
for me (and if the patch is fine for you and it gets included, Fedora
will automatically benefit from this, too, once they update to a new
version of grub).

 Regards,
  Thomas