Subject: Re: New wsdisplay for playstation2
To: None <M.Drochner@fz-juelich.de>
From: Jorge Acereda =?iso-8859-1?q?Maci=E1?= <al004046@anubis.uji.es>
List: tech-kern
Date: 10/28/2003 19:17:40
Matthias Drochner <M.Drochner@fz-juelich.de> writes:

> al004046@anubis.uji.es said:
> > gsfb_mmap(void *v, off_t offset, int prot) {
> > 	if (GSFB_BUFFER_SIZE >= (unsigned) offset)
> > 	    return NULL; 
> 
> I didn't look deeper, but this piece of code is wrong for two
> reasons:
> 1. the comparision is reverted
> 2. returning NULL doesn't what you expect - probably you get
>    the physical page at 0 mapped
>    to return an error, use (-1)

Oops, thanks a lot!

BTW, I "inherited" the "return NULL;" from the previous
arch/playstation2/ee/gsfb.c, so I guess it is also buggy.