[LU-8802] Dynamically allocate obd_devices.

• Status: Open
• Project: Lustre
• Type: Improvement
• Priority: Minor
• Reporter: James A Simmons
• Assignee: James A Simmons
• Resolution: Unresolved

Description

One proposed cleanup upstream was removal of class_devno_max(). It was pointed out that was for future work to enable dynamic obd_device allocation.
Aya Mahfouz has stepped forward wanting to do this work. Created this ticket to keep track of the progress and have reviewers look over his work.

Andreas replied to Aya's email with the following suggestions:

There are several ways to approach this problem to make the allocation of the obd_devs[] array dynamic. In most cases, there isn't any value to dynamically shrink this array, since the filesystem(s) will typically be mounted until the node is rebooted, and it is only in the tens of KB size range, so this will not affect ongoing operations, and that simplifies the implementation.

The easiest way would be to have a dynamically-sized obd_devs[] array that is reallocated in class_newdev() in PAGE_SIZE chunks whenever the current array has no more free slots and copied to the new array, using obd_dev_lock to protect the array while it is being reallocated and copied. In most cases, this would save memory over the static array (not many filesystems have so many servers), but for the few sites that have 10000+ servers, and/or mounting multiple large filesystems on a single node, they don't need to change the source to handle this. Using libcfs_kvzalloc() would avoid issues with allocating large chunks of memory.

There are a few places where obd_devs[] is accessed outside obd_dev_lock that would need to be fixed now that this array may be changed at runtime.

A second approach that may scale better is to change obd_devs from an array to a doubly linked list (using standard list_head helpers). In many cases the whole list is seached linearly in the cases where a hash table lookup for the device name/UUID is not used, and most of the uses of class_num2obd() are just used to walk that list in order, which could be replaced with list_for_each_entry() list traversal. The class_name2dev() function should be changed to return the pointer to struct obd_device, and a new helper class_dev2num() would just return the obd_minor number from struct obd_device for the one use in class_resolve_dev_name(). Using a linked list has the advantage that there is no need to search for free slots in the array, since devices would be removed from the list when it is freed.

Issue Links

• is duplicated by: LU-147 avoid 8k obd device amount limit

Comments

[Peter Jones]

James

It probably makes sense fpr you to own this ticket as you will likely be reviewing it when ready

Peter

[Andreas Dilger]

The patch https://review.whamcloud.com/347 "LU-147 avoid 8k obd device amount limit" used a different mechanism to remove this limit, but had to be reverted due to defects. It would probably be useful to review, even if the final approach is different.

[Andreas Dilger]

It turns out that containers/VMs on clients like to mount the same filesystems many times, so that may drive some interest in reviving this work, instead of the need to support a single 10000-OST filesystem mount.

[Gerrit Updater]

"Timothy Day <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51040
Subject: LU-8802 obd: remove MAX_OBD_DEVICES
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: eead8e7f0d6830d5a63d59541888ec8d20b5e2fb

[Gerrit Updater]

"Timothy Day <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51920
Subject: LU-8802 obd: add strong locking to OBD lifecycle management
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 73c2c71a49c9e079dfa0273256d34e83ba18651f

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/51040/
Subject: LU-8802 obd: remove MAX_OBD_DEVICES
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: c5e5060d9502c2e4fb43579d02c983ccbec9b063

[Gerrit Updater]

"Timothy Day <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53466
Subject: LU-8802 obd: str2dev is missing obd_device_unlock()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: a88d2e89de0737e56bb1ff03d368f3f7838b0ffd

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/53466/
Subject: LU-8802 obd: str2dev is missing obd_device_unlock()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: c68fce096fc1eadfb6186f57e4c3b2b239392c6e

[Gerrit Updater]

"Timothy Day <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/53606
Subject: LU-8802 obd: add uuid and name hash tables
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: bdeb4a5ce22e4d91e978d128ea328f21acaa2185