[syslinux] [PATCH 2/2] core/graphics: fix lss16 parsing

H. Peter Anvin hpa at zytor.com
Tue Sep 1 15:16:37 PDT 2015


On 08/23/2015 12:33 AM, Patrick Masotta via Syslinux wrote:
> 
>>>>
>  Unfortunately,
>  that other code is a massive single patch and would be hard to review
>  since it would be difficult to bisect should a bug be introduced.  I
>  will try to split the other patch into logical parts and submit as time
>  permits.
>  <<<
> 
> Yes I know; unfortunately solving the lss16 issue cannot be done with a couple of 4 line patches. 
> The code to fix was originally ported to C from the assembler of a previous SL version, and it 
> never really worked before; it was needed re-writing certain parts.
> Now the new code ( http://www.syslinux.org/archives/2015-July/023835.html ) has been 
> tested and it does work (you can see an lss16 image in your terminal). I always prefer start 
> fixing things from a working code base than start patching one that never did.
> 
> if you want to continue with the task probably it's more valuable if you try to tackle
> the associated issues related to terminal/cli handling rather that splitting the big patch.
> just my 2 cents.
> 

I do want to emphasize what I have already said, which is that I would
prefer for this to be done via the VESA subsystem rather than doing
low-level hacking with the VGA display.  One reason is that the quality
of video BIOS continues to drop, and we see more and more bugs in text
output on a graphics screen since no modern OS uses it, and for non-BIOS
it doesn't work at all.  Note that the VESA subsystem can already decode
LSS16.  However, at the same time it is obviously clear that users can't
wait forever.

	-hpa



More information about the Syslinux mailing list