[PATCH][RFC] FDI mmap optimization

Danny Kukawka danny.kukawka at web.de
Wed Dec 13 08:25:11 PST 2006


On Wednesday 13 December 2006 16:54, Sergey Lapin wrote:
> Hi,
> please see attached patch, it frees a few kbs of heap on my setup
> (embedded ARM device) and reduces memory fragmentation.

How much bring the patch? Any statistics?

Anyway, on the first lock at the patch I would say this part is not the best 
idea:
 +       if((tmpdir = getenv("TMPDIR")) == NULL) {
 +           tmpdir = "/tmp";
 +       }
 +       cachename = g_strconcat(tmpdir, "/fdi_cache", NULL);

IMO hal should create such a file better e.g. under /var/run/hal/ or /var/tmp

Danny



More information about the hal mailing list