Opened 11 years ago

Closed 22 months ago

#2918 closed bug (not reproducible)

on screen debug: many keystrokes needed to get to next page of messages

Reported by: Adek336 Owned by: nobody
Priority: normal Milestone: R1
Component: System/Boot Loader Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

When on screen debug is selected at boot time, I have to press a key to see more and more pages of messages. However about the time AHCI messages appear, suddenly I need a lot of keystrokes to get the next pages of messages. It doesn't seem like this is just a matter of the kernel not issueing any messages for a significant time: the "Press any key to continue" text doesn't disappear, and when it finally does print the next page of messages they all come very quickly.

Attachments (1)

kbd.1.diff (2.8 KB ) - added by Adek336 11 years ago.
proof of concept

Download all attachments as: .zip

Change History (12)

comment:1 by PieterPanman, 11 years ago

See also #3129, I have the same thing. Is anyone NOT having this? Maybe it is due to some hardware component?

by Adek336, 11 years ago

Attachment: kbd.1.diff added

proof of concept

comment:2 by Adek336, 11 years ago

The interrupt handler debug_keyboard_interrupt() steals the keystroke from arch_debug_blue_screen_getchar() in x86 (perhaps this also affects other architectures).

To fix the problem, arch_debug_blue_screen_getchar() has to force the isr to not read from ps2 port, temporarily uninstall the isr, clear interrupts or mask the keyboard irq. kbd.1.diff is an example fix.

Also note that F12 / BREAK key isn't programmed to open the KDL, when pressed against a "Press key to continue" message.

comment:3 by PieterPanman, 11 years ago

I can confirm your patch works on my laptop, it now responds perfectly. Very nice.

comment:4 by Adek336, 11 years ago

Does the shift key work in the debugger for you with the patch applied?

comment:5 by PieterPanman, 11 years ago

Nope it does not. I also cannot enter KDL using F12 while it is paging the debugging output :) It just goes to the next page...

comment:6 by scottmc, 8 years ago

any progress on this one, it is still an issue? Perhaps you can verify that the patch still applies cleanly on a recent build of Haiku and double check for proper Haiku coding style. I believe the returns get their own line... there's probably a few other issues in there as well.

comment:7 by diver, 7 years ago

Has a Patch: set

comment:8 by diver, 7 years ago

Component: - GeneralSystem/Boot Loader
Version: R1/pre-alpha1R1/Development

comment:9 by pulkomandy, 3 years ago

Can anyone still reproduce this? (note: you need a PS/2 keyboard - USB won't have the same problem)

Last edited 3 years ago by pulkomandy (previous) (diff)

comment:10 by axeld, 2 years ago

Owner: changed from axeld to nobody
Status: newassigned

comment:11 by waddlesplash, 22 months ago

Resolution: not reproducible
Status: assignedclosed

No reply in 13 months, closing

Note: See TracTickets for help on using tickets.