Attachment 362793 Details for Bug 603161

[patch] seabios patch that fixes this issue

enable-irqs-in-spun-on-calls.patch (text/plain), 1.45 KB, created by Bruce Rogers on 2010-05-17 22:50:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Bruce Rogers

Created: 2010-05-17 22:50:06 UTC

Size: 1.45 KB

patch

obsolete

>commit 68c51390e3c78ed4a5b012d9097007f7cb0b8bbe
>Author: Kevin O'Connor <kevin@koconnor.net>
>Date:   Sat Mar 13 22:23:44 2010 -0500
>
>    Enable irqs in kbd/clock calls that caller might "spin" on.
>    
>    Some old programs will spin on a clock/keyboard call with irqs
>    disabled.  They assume the BIOS will enable irqs and allow key events
>    and clock events to occur.
>    
>    So, enable irqs in those functions that a caller might "spin" on.
>
>diff --git a/src/clock.c b/src/clock.c
>index 5a30e35..9afa71d 100644
>--- a/src/clock.c
>+++ b/src/clock.c
>@@ -226,6 +226,7 @@ timer_setup(void)
> static void
> handle_1a00(struct bregs *regs)
> {
>+    yield();
>     u32 ticks = GET_BDA(timer_counter);
>     regs->cx = ticks >> 16;
>     regs->dx = ticks;
>diff --git a/src/kbd.c b/src/kbd.c
>index 6f3ae15..36c89fc 100644
>--- a/src/kbd.c
>+++ b/src/kbd.c
>@@ -73,6 +73,7 @@ enqueue_key(u8 scan_code, u8 ascii_code)
> static void
> dequeue_key(struct bregs *regs, int incr, int extended)
> {
>+    yield();
>     u16 buffer_head;
>     u16 buffer_tail;
>     for (;;) {
>@@ -126,6 +127,7 @@ handle_1601(struct bregs *regs)
> static void
> handle_1602(struct bregs *regs)
> {
>+    yield();
>     regs->al = GET_BDA(kbd_flag0);
> }
> 
>@@ -184,6 +186,7 @@ handle_1611(struct bregs *regs)
> static void
> handle_1612(struct bregs *regs)
> {
>+    yield();
>     regs->al = GET_BDA(kbd_flag0);
>     regs->ah = ((GET_BDA(kbd_flag1) & ~(KF2_RCTRL|KF2_RALT))
>                 | (GET_BDA(kbd_flag2) & (KF2_RCTRL|KF2_RALT)));

Actions: View | Diff

Attachments on bug 603161: 362793