firewire: core: fill model field in modalias of unit device for legacy layout of configuration ROM

As the last part of support for legacy layout of configuration ROM, this
commit traverses vendor directory as well as root directory when
constructing modalias for unit device. The change brings loss of backward
compatibility since it can fill model field ('mo') which is 0 at current
implementation in the case. However, we can be optimistic against
regression for unit drivers in kernel, due to some points:

1. ALSA drivers for audio and music units use the model fields to match
   device, however all of supported devices does not have such legacy
   layout.
2. the other unit drivers (e.g. sbp2) does not use the model field to
   match device.

The rest of concern is user space application. The most of applications
just take care of node device and does not use the modalias of unit
device, thus the change does not affect to them. But systemd project is
known to get affects from the change since it includes hwdb to take udev
to configure fw character device conveniently. I have a plan to work for
systemd so that the access permission of character device could be kept
across the change.

Suggested-by: Adam Goldman <adamg@pobox.com>
Link: https://lore.kernel.org/r/20231221134849.603857-9-o-takashi@sakamocchi.jp
Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
This commit is contained in:
Takashi Sakamoto 2023-12-25 07:23:01 +09:00
parent 2eab8bc0f0
commit 986c20bb3e
2 changed files with 20 additions and 3 deletions

View File

@ -153,8 +153,25 @@ static void get_ids(const u32 *directory, int *id)
static void get_modalias_ids(const struct fw_unit *unit, int *id)
{
get_ids(&fw_parent_device(unit)->config_rom[ROOT_DIR_OFFSET], id);
get_ids(unit->directory, id);
const u32 *root_directory = &fw_parent_device(unit)->config_rom[ROOT_DIR_OFFSET];
const u32 *directories[] = {NULL, NULL, NULL};
const u32 *vendor_directory;
int i;
directories[0] = root_directory;
// Legacy layout of configuration ROM described in Annex 1 of 'Configuration ROM for AV/C
// Devices 1.0 (December 12, 2000, 1394 Trading Association, TA Document 1999027)'.
vendor_directory = search_directory(root_directory, CSR_VENDOR);
if (!vendor_directory) {
directories[1] = unit->directory;
} else {
directories[1] = vendor_directory;
directories[2] = unit->directory;
}
for (i = 0; i < ARRAY_SIZE(directories) && !!directories[i]; ++i)
get_ids(directories[i], id);
}
static bool match_ids(const struct ieee1394_device_id *id_table, int *id)

View File

@ -178,7 +178,7 @@ static void device_attr_legacy_avc(struct kunit *test)
};
struct device *node_dev = (struct device *)&node.device;
struct device *unit0_dev = (struct device *)&unit0.device;
static const int unit0_expected_ids[] = {0x00012345, 0x00000000, 0x00abcdef, 0x00543210};
static const int unit0_expected_ids[] = {0x00012345, 0x00fedcba, 0x00abcdef, 0x00543210};
char *buf = kunit_kzalloc(test, PAGE_SIZE, GFP_KERNEL);
int ids[4] = {0, 0, 0, 0};