Hi Frederic, On Thu, Sep 27, 2012, Frederic Danis wrote: > Here is my proposal for new storage directory structure using > ini-file format. > > Each adapter directory (/var/lib/bluetooth/<adapter address>/) will > contain a config file for the local adapter and one directory per > remote device. > The adapter config file just need to be converted to ini-file format > with only 1 group called [adapter]. > > Each of remote device directories' name will be based on remote > device address and address type (address#type). > This directory will contain a config file with remote device infos > and a linkkey file. > Remote device config file will include a [device] group with general > device infos (name, alias, profiles or services list, ...), and > groups named by profile uuid (or service uuid) with related infos. > > So the directory structure should be: > /var/lib/bluetooth/<adapter address>/ > ./config > ./<remote device address#type>/ > ./config > ./linkkey > ./<remote device address#type>/ > ./config > ./linkkey > ... So far this all looks good, though maybe we should follow the convention of having .conf suffixes like with our other INI files. Or maybe that's not needed? > I attached sample of adapter and device config files. > [adapter] > name=desktop-0 > class=0x780011 > pairable=yes > onmode=discoverable > mode=discoverable I hope you've looked at our existing INI files like audio.conf and main.conf. You should have noticed that we use CamelCase for the section and variable names, so at least that needs fixing. > [device] > name=MyPhone > alias=Fred's phone > class=0x180204 > device_id=FFFF 0000 0000 0000 > eir=040D040218 > manufacturer=15 > lmp_version=2 > lmp_subversion=777 > features=FFFE0D0008080000 > lastseen=2012-09-26 11:19:40 GMT > lastused=2012-09-26 11:43:42 GMT > trusted=yes > profiles=00001101-0000-1000-8000-00805f9b34fb;00001103-0000-1000-8000-00805f9b34fb > > [00001101-0000-1000-8000-00805f9b34fb] > handle=10001 > record=35470900000A000100010900013503191101090004350C350319010035051900030802090005350319100209000935083506191101090100090100250C53657269616C20506F727400 > > [00001103-0000-1000-8000-00805f9b34fb] > handle=10002 > record=35530900000A000100020900013503191103090004350C35031901003505190003080309000535031910020900093508350619110309010009010025134469616C2D7570204E6574776F726B696E67000903052800 This looks ok too, except for the lack of CamelCase naming. One of the most interesting files I was waiting to see is the linkkey one since in the existing storage we've crammed lots of separate variables into the same entry. Could you send a proposal for this too in your next revision? Since we also need storage for LTK's maybe it'd make sense to have a single "keys" file with [LinkKey] and [LongTermKey] sections? Or do you think those should be separate files? Also, it seems you've left out all LE-specific information from this initial proposal, like the conversion of the existing primaries file. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
