[PATCH] XKB: cache xkbcomp output for fast start-up v3 for 1.6.0

Li, Yan yanli at infradead.org
Wed Apr 8 00:22:39 PDT 2009


On Fri, Apr 03, 2009 at 07:55:34AM -0700, Alan Coopersmith wrote:
> Is there a race condition if two servers start at once (two vt's or
> multiseat, etc.) and the first server has started writing the .xkm
> but not finished when the second one finds it and tries to read?
> (Perhaps it should write to a temporary name and then rename() when
>  writing is complete.)    We had problems with that with the old
> code before we switched to exclusively using server-<display>.xkm
> naming.

Thanks for pointing this out.  Using a temp file and then rename() is
OK for me.  I'll post a new patch to fix this soon.

> I'd also rather see the Sha1 helper function moved to a file under
> the os dir and shared by both render/glyph.c & this code instead
> of duplicating the code here and giving people two places to make
> the same updates when adding new sha1 library implementations.

Sure.  But how about putting this into another patch (I can make it),
since I don't know how to test the change to render/glyph.c. By
putting this into another patch we can test the impact to
render/glyph.c separately.

-- 
Li, Yan



More information about the xorg mailing list