Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#14942 closed enhancement (fixed)

hrev52910 disables nVidia VESA scaling - regression

Reported by: smallstepforman Owned by: mmu_man
Priority: normal Milestone: R1/beta2
Component: System/Boot Loader Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
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 6 years ago.
Haiku postage stamp
IMG_6501.jpg (329.4 KB ) - added by smallstepforman 6 years ago.
Haiku postage stamp

Download all attachments as: .zip

Change History (9)

by smallstepforman, 6 years ago

Attachment: IMG_6500.JPG added

Haiku postage stamp

by smallstepforman, 6 years ago

Attachment: IMG_6501.jpg added

Haiku postage stamp

comment:1 by waddlesplash, 6 years ago

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 by mmu_man, 6 years ago

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 by waddlesplash, 6 years ago

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

comment:4 by mmu_man, 6 years ago

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

comment:5 by mmu_man, 6 years ago

As of hrev53109 it is now an option in the vesa settings file.

comment:6 by mmu_man, 6 years ago

Resolution: fixed
Status: assignedclosed

So I guess we can close it now.

comment:7 by nielx, 5 years ago

Milestone: UnscheduledR1/beta2

Assign tickets with status=closed and resolution=fixed within the R1/beta2 development window to the R1/beta2 Milestone

Note: See TracTickets for help on using tickets.