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

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

Posted by Gerd Hoffmann 5 years, 8 months ago

Fixes console redirection for NetBSD primary bootloader.

https://bugs.launchpad.net/bugs/1743191
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 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);
     }
 }
-- 
2.18.2

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

Posted by Kevin O'Connor 5 years, 8 months ago

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 <kraxel@redhat.com>
> ---
>  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@koconnor.net>

-Kevin
_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org

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

Posted by Paul Menzel 5 years, 8 months ago

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-com.iso
    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 <kraxel@redhat.com>
> ---
>  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 <pmenzel@molgen.mpg.de>


Kind regards,

Paul

_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org