Re: [PATCH] Add compressed ID database support to (usb|pci)-db

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 29.09.09 19:04, David Zeuthen (david@xxxxxxxx) wrote:

> FWIW, this thing came up a couple of years ago on the hal list and back
> then I very strongly refused to support compressed ids files. Mostly on
> the grounds that it makes it impossible to mmap the files. E.g. you need
> to allocate memory for the uncompressed image.
> 
> In udev this is even worse because we'd be doing all this work over and
> over again - for every freaking event. In HAL, we mmaped the ids files
> at start-up, built a simple look-up table and did binary lookups on
> every event.
> 
> Just so you know, I'm still very much against this and I think it would
> be a mistake to support compressed ids files in in udev. Please don't do
> it.

Ok, that makes sense to me. Sorry, Piter, it is unlikely that we will
merge your patch.

Even though your work will not be merged it is really appreciated!

Sorry,

Lennart

-- 
Lennart Poettering                        Red Hat, Inc.
lennart [at] poettering [dot] net
http://0pointer.net/lennart/           GnuPG 0x1A015CC4
--
To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [Linux DVB]     [Asterisk Internet PBX]     [DCCP]     [Netdev]     [X.org]     [Util Linux NG]     [Fedora Women]     [ALSA Devel]     [Linux USB]

  Powered by Linux