[Fontconfig] icon theme weight query ... (fwd)

[Behdad Esfahbod]

Hi,

Michael Meeks points a very inefficient read loop in fontconfig.
Should be easy to fix...  Would have been easier in fact if
fontconfig was mmapping the cache file right away...

--behdad
http://behdad.org/

"Commandment Three says Do Not Kill, Amendment Two says Blood Will Spill"
	-- Dan Bern, "New American Language"

---------- Forwarded message ----------
Date: Mon, 6 Feb 2006 05:45:50 -0500
From: michael meeks <michael.meeks at novell.com>
To: performance-list at gnome.org
Cc: Rodrigo Moya <rodrigo at novell.com>
Subject: icon theme weight query ...

Hi guys,

	Just running gnome-screenshot-applet and seeing some rather nasty
performance problems (some of which are most likely related to some
fontconfig grief we've been suffering recently)[1]

		Michael.
[snip]

[1] - I though only OO.o did things as stupid as this:

8844  stat64("/home/michael/.fonts.cache-2", {st_mode=S_IFREG|0600,
st_size=217089, ...}) = 0
8844  open("/home/michael/.fonts.cache-2", O_RDONLY) = 23
8844  read(23, "G", 1)                  = 1
8844  read(23, "L", 1)                  = 1
8844  read(23, "O", 1)                  = 1
8844  read(23, "B", 1)                  = 1
8844  read(23, "A", 1)                  = 1
8844  read(23, "L", 1)                  = 1
8844  read(23, "\0", 1)                 = 1
8844  lseek(23, 0, SEEK_SET)            = 0
8844  read(23, "G", 1)                  = 1
8844  read(23, "L", 1)                  = 1
8844  read(23, "O", 1)                  = 1
8844  read(23, "B", 1)                  = 1
8844  read(23, "A", 1)                  = 1
8844  read(23, "L", 1)                  = 1
8844  read(23, "\0", 1)                 = 1
8844  lseek(23, 0, SEEK_CUR)            = 7
8844  lseek(23, 7, SEEK_SET)            = 7
8844  read(23, " ", 1)                  = 1
8844  read(23, " ", 1)                  = 1
8844  read(23, " ", 1)                  = 1
8844  read(23, "3", 1)                  = 1
8844  read(23, "4", 1)                  = 1
8844  read(23, "f", 1)                  = 1
8844  read(23, "f", 1)                  = 1
8844  read(23, "9", 1)                  = 1
8844  read(23, " ", 1)                  = 1
8844  read(23, "7", 1)                  = 1
8844  read(23, "8", 1)                  = 1
8844  read(23, "5", 1)                  = 1
8844  read(23, "6", 1)                  = 1
8844  read(23, "3", 1)                  = 1
8844  read(23, "4", 1)                  = 1
8844  read(23, "1", 1)                  = 1
8844  read(23, "2", 1)                  = 1
8844  read(23, " ", 1)                  = 1
8844  read(23, " ", 1)                  = 1
8844  read(23, " ", 1)                  = 1
8844  read(23, " ", 1)                  = 1
8844  read(23, "1", 1)                  = 1
....

-- 
 michael.meeks at novell.com  <><, Pseudo Engineer, itinerant idiot

_______________________________________________
Performance-list mailing list
Performance-list at gnome.org
http://mail.gnome.org/mailman/listinfo/performance-list