Messages in this thread | | | Date | Fri, 29 Jul 2005 14:58:48 -0400 | From | Dave Jones <> | Subject | Re: vesafb-fix-mtrr-bugs.patch added to -mm tree |
| |
On Fri, Jul 29, 2005 at 11:24:37AM -0700, Andrew Morton wrote:
> From: "Antonino A. Daplas" <adaplas@gmail.com> > > >> vesafb: mode is 800x600x16, linelength=1600, pages=16 > >> vesafb: scrolling: redraw > >> vesafb: Truecolor: size=0:5:6:5, shift=0:11:5:0 > >> mtrr: type mismatch for fc000000,1000000 old: write-back new: write- > >> combining > > Range is already set to write-back, vesafb attempts to add a write-combining > mtrr (default for vesafb). > > >> mtrr: size and base must be multiples of 4 kiB > > This is a bug, vesafb attempts to add a size < PAGE_SIZE triggering > the messages below.
I fixed this a few weeks back. It's this line which your patch removes..
- while (temp_size > PAGE_SIZE &&
> To eliminate the warning messages, you can add the option mtrr:2 to add a > write-back mtrr for vesafb. Or just use nomtrr option.
If we need users to pass extra command line args to make warnings go away, we may as well not bother. Because 99% of users will be completely unaware that option even exists. They'll still see the same message, and still report the same bugs.
The pains of MTRR strike again. This stuff is just screaming for a usable PAT implementation. Andi, you were working on that, any news ? Or should I resurrect Terrence's patch again ?
Dave
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |