Opened 8 weeks ago

Last modified 8 weeks ago

#14942 assigned enhancement

hrev52910 disables nVidia VESA scaling - regression

Reported by: smallstepforman Owned by: mmu_man
Priority: normal Milestone: Unscheduled
Component: System/Boot Loader Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

hrev52910 disabled nVidia VESA scaling, and the result is arguably not better than the previous version. Using Haiku in a postage stamp window cannot be better than running it full screen at a miscaled resolution, expecially on modern Hi-DPI displays. See attachment. One could argue that the previous version was many times better.

Attachments (2)

IMG_6500.JPG (329.4 KB) - added by smallstepforman 8 weeks ago.
Haiku postage stamp
IMG_6501.jpg (329.4 KB) - added by smallstepforman 8 weeks ago.
Haiku postage stamp

Download all attachments as: .zip

Change History (6)

Changed 8 weeks ago by smallstepforman

Attachment: IMG_6500.JPG added

Haiku postage stamp

Changed 8 weeks ago by smallstepforman

Attachment: IMG_6501.jpg added

Haiku postage stamp

comment:1 Changed 8 weeks ago by waddlesplash

Component: Drivers/Graphics/VESASystem/Boot Loader
Keywords: VESA scaling removed
Owner: changed from nobody to mmu_man
Status: newassigned

Indeed, it seems this is worse.

Probably the commit should just be reverted then. mmu_man will have to just implement VESA BIOS patching to get real modesetting if he wants it... :P

comment:2 Changed 8 weeks ago by mmu_man

No need for two photos…

Well, Yes and now, here with scaling it distorts the aspect ratio and it's really ugly. Without it I do have a full height display but with black borders on the side, and it's definitely better. I suppose it depends on the age of your VESA BIOS.

I tried various arguments to the call, some are supposed to scale but keep the aspect ratio, but it didn't work for me.

I could make it an option I suppose.

comment:3 Changed 8 weeks ago by waddlesplash

I don't think we can use options at this stage. Probably we should just implement VESA BIOS livepatching.

comment:4 Changed 8 weeks ago by mmu_man

Well, the video code in the bootloader does check the vesa settings for the video mode, so it should be possible.

Note: See TracTickets for help on using tickets.