[PATCH] sercon: vbe modeset is int 10h function 4f02 not 4f00

[Gerd Hoffmann]

Fixes console redirection for NetBSD primary bootloader.

https://bugs.launchpad.net/bugs/1743191
Signed-off-by: Gerd Hoffmann 
---
 src/sercon.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/sercon.c b/src/sercon.c
index a5dadb7261af..66a1f2412e77 100644
--- a/src/sercon.c
+++ b/src/sercon.c
@@ -464,7 +464,7 @@ static void sercon_104f(struct bregs *regs)
         regs->ax = 0x0100;
     } else {
         // Disable sercon entry point on any vesa modeset
-        if (regs->al == 0x00)
+        if (regs->al == 0x02)
             SET_LOW(sercon_enable, 0);
     }
 }

[Paul Menzel]

Dear Gerd,

Maybe in imperative mood:
> Correct VBE modeset to int 10h function 4f02 not 4f00

On 2020-03-06 10:03, Gerd Hoffmann wrote:
> Fixes console redirection for NetBSD primary bootloader.

Add how to reproduce this:

wget http://ftp.netbsd.org/pub/NetBSD/NetBSD-7.1.1/amd64/installation/cdrom/boot-...
    qemu-system-x86_64 -nographic -cdrom boot-com.iso

For backports add:

Fixes: d6728f30 ("add serial console support")
> https://bugs.launchpad.net/bugs/1743191
> Signed-off-by: Gerd Hoffmann
>
> src/sercon.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/sercon.c b/src/sercon.c
> index a5dadb7261af..66a1f2412e77 100644
> --- a/src/sercon.c
> +++ b/src/sercon.c
> @@ -464,7 +464,7 @@ static void sercon_104f(struct bregs *regs)
>          regs->ax = 0x0100;
>      } else {
>          // Disable sercon entry point on any vesa modeset
>    if (regs->al == 0x00)
>    if (regs->al == 0x02)
>        SET_LOW(sercon_enable, 0);
> }
>
> }

Reviewed-by: Paul Menzel

Kind regards,

Paul

[Kevin O'Connor]

On Fri, Mar 06, 2020 at 10:03:21AM +0100, Gerd Hoffmann wrote:
> Fixes console redirection for NetBSD primary bootloader.
>
> https://bugs.launchpad.net/bugs/1743191
> Signed-off-by: Gerd Hoffmann
>
> src/sercon.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/sercon.c b/src/sercon.c
> index a5dadb7261af..66a1f2412e77 100644
> --- a/src/sercon.c
> +++ b/src/sercon.c
> @@ -464,7 +464,7 @@ static void sercon_104f(struct bregs *regs)
>          regs->ax = 0x0100;
>      } else {
>          // Disable sercon entry point on any vesa modeset
>    if (regs->al == 0x00)
>    if (regs->al == 0x02)
>        SET_LOW(sercon_enable, 0);
> }
>
> }

Reviewed-by: Kevin O'Connor

-Kevin