From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Antonino A. Daplas" Subject: Re: [Linux-fbdev-devel] [PATCH 1/1 2.6.13] framebuffer: bit_putcs() optimization for 8x* fonts Date: Wed, 31 Aug 2005 14:42:36 +0800 Message-ID: <4315515C.1040208@gmail.com> References: <43148610.70406@t-online.de> <43149E5B.7040006@t-online.de> <4314DD2E.7060901@t-online.de> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: Cc: Knut Petersen , linux-fbdev-devel@lists.sourceforge.net, Andrew Morton , Linux Kernel Development , Jochen Hein Roman Zippel wrote: > Hi, > > On Wed, 31 Aug 2005, Knut Petersen wrote: > >> How could I make it an inline function? It is used in console/bitblit.c, >> nvidia/nvidia.c, >> riva/fbdev.c and softcursor.c. > > Something like below, which has the advantange that there is still only > one implementation of the function and if it's still slower, we really > need to check the compiler. > I do get better numbers with this, not much, but better than Knut's (5ms), and definitely much better than the old uninlined one (100ms). Andrew, don't get this yet. I'll incorporate this with the bit_putcs() breakup that I'm currently doing. Roman, okay if you have a 'Signed-off-by' line? Tony