Re: [PATCH 2.4.21-rc1] vesafb with large memory

Willy Tarreau
2003-05-03T21:46:22Z

On Sat, May 03, 2003 at 03:46:57PM +0300, Thomas Backlund wrote:

> Oh man...
> I must have been sleeping when I posted that patch...

no problem, we all need some sleep at least once a week ;-)

> the correct line should AFAIK be:
> video_size = screen_info.lfb_width * screen_info.lfb_height * video_bpp;
>
> (AFAIK we are calculating bits here, not bytes so the '/8' you used is
> wrong... could you try without it, and let me know...)

OK, I was assuming that video_size was in bytes. I will retry without the /8.

> I'll be rebuilding and retesting my system today or tommorrow,
> but I wuold like to hear if it works for you...

I just come back home this evening, so let's say you'll get a feedback tomorrow
morning.

Regards,
Willy