[PATCH] client_stack(): Only stack windows once per mainloop

2009-04-10 Thread Uli Schlachter
I was creating 2000 wiboxes in a loop (don't ask) and creating them took forever. According to callgrind, there were about 2 million calls to xcb_configure_window() and most (if not all) of them were from client_stack(). Awesome spent 70% of its cpu time in these client_stack() calls.

Re: [PATCH] client_stack(): Only stack windows once per mainloop

2009-04-10 Thread Julien Danjou
At 1239357411 time_t, Uli Schlachter wrote: This patch makes client_stack() only set a flag which is later checked. This should reduce the number of restacks to the bare minimum. With this patch, neither xcb_configure_window() nor anything else client_stack() related shows up as having a lot

Re: [PATCH] client_stack(): Only stack windows once per mainloop

2009-04-10 Thread Uli Schlachter
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Julien Danjou wrote: At 1239378506 time_t, Uli Schlachter wrote: Here is the new version of the patch, rebased on top of latest master. Pushed. PS: I'd suggest you publish a branch online so I can merge, since you are sending a lot of patch.