From: Paolo Ornati <ornati@fastwebnet.it>
To: adaplas@pol.net
Subject: Re: 2.6.9-rc1: scrolling with tdfxfb 5 times slower
Date: Wed, 1 Sep 2004 09:20:13 +0200
User-Agent: KMail/1.6.2
Cc: linux-kernel@vger.kernel.org, "David S. Miller" <davem@davemloft.net>
References: <200408312133.40039.ornati@fastwebnet.it> <200409010729.07156.adaplas@hotpop.com>
In-Reply-To: <200409010729.07156.adaplas@hotpop.com>
MIME-Version: 1.0
Content-Disposition: inline
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Message-Id: <200409010920.13307.ornati@fastwebnet.it>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2996
Lines: 89

On Wednesday 01 September 2004 01:29, Antonino A. Daplas wrote:
> On Wednesday 01 September 2004 03:33, Paolo Ornati wrote:
> > Tests with "linux/MAINTAINERS" (time cat MAINTAINERS)
> > 2.6.8.1
> > 	real    0m2.625s
> > 	user    0m0.000s
> > 	sys     0m2.621s
> >
> > 2.6.9-rc1
> > 	real    0m13.528s
> > 	user    0m0.000s
> > 	sys     0m13.553s
> >
> >
> > Also many -mm kernels are affected... this is obvious because these
> > patches come from Andrew's tree ;-)
>
> Try this patch also, but select Y for CONFIG_FB_3DFX_ACCEL.  You also
> need to revert info->flags to FBINFO_DEFAULT | FBINFO_HWACCEL_YPAN;
>
> Tony
>
> diff -uprN linux-2.6.9-rc1-mm1-orig/drivers/video/tdfxfb.c
> linux-2.6.9-rc1-mm1/drivers/video/tdfxfb.c ---
> linux-2.6.9-rc1-mm1-orig/drivers/video/tdfxfb.c	2004-08-30
> 19:39:19.000000000 +0800 +++
> linux-2.6.9-rc1-mm1/drivers/video/tdfxfb.c	2004-09-01 07:23:12.855078672
> +0800 @@ -168,7 +168,6 @@ static int banshee_wait_idle(struct fb_i
>  static void tdfxfb_fillrect(struct fb_info *info, const struct
> fb_fillrect *rect); static void tdfxfb_copyarea(struct fb_info *info,
> const struct fb_copyarea *area); static void tdfxfb_imageblit(struct
> fb_info *info, const struct fb_image *image); -static int
> tdfxfb_cursor(struct fb_info *info, struct fb_cursor *cursor); #endif /*
> CONFIG_FB_3DFX_ACCEL */
>
>  static struct fb_ops tdfxfb_ops = {
> @@ -183,13 +182,12 @@ static struct fb_ops tdfxfb_ops = {
>  	.fb_fillrect	= tdfxfb_fillrect,
>  	.fb_copyarea	= tdfxfb_copyarea,
>  	.fb_imageblit	= tdfxfb_imageblit,
> -	.fb_cursor	= tdfxfb_cursor,
>  #else
>  	.fb_fillrect	= cfb_fillrect,
>  	.fb_copyarea	= cfb_copyarea,
>  	.fb_imageblit	= cfb_imageblit,
> -	.fb_cursor	= soft_cursor,
>  #endif
> +	.fb_cursor	= soft_cursor,
>  };
>
>  /*
> @@ -1030,7 +1028,9 @@ static void tdfxfb_imageblit(struct fb_i
>  	}
>  	banshee_wait_idle(info);
>  }
> +#endif /* CONFIG_FB_3DFX_ACCEL */
>
> +#ifdef TDFX_HARDWARE_CURSOR
>  static int tdfxfb_cursor(struct fb_info *info, struct fb_cursor *cursor)
>  {
>  	struct tdfx_par *par = (struct tdfx_par *) info->par;
> @@ -1167,7 +1167,7 @@ static int tdfxfb_cursor(struct fb_info
>  	spin_unlock_irqrestore(&par->DAClock, flags);
>  	return 0;
>  }
> -#endif /* CONFIG_FB_3DFX_ACCEL */
> +#endif
>
>  /**
>   *      tdfxfb_probe - Device Initializiation

Ok, with this patch and CONFIG_FB_3DFX_ACCEL=y the scrolling speed comes 
back (only a bit slower than with 2.6.8.1 without CONFIG_FB_3DFX_ACCEL):

$ time cat MAINTAINERS: ~2.67s

Another interesting thing is that if I enable CONFIG_FB_3DFX_ACCEL without 
your patch the screen becomes black and the kernel stop working at boot 
time (when the mode switch happens).

-- 
	Paolo Ornati
	Gentoo Linux (kernel 2.6.8-gentoo-r3)

-
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/