[LU-11133] lustre-ldiskfs-dkms package does not provide the ldiskfs kernel module Created: 09/Jul/18 Updated: 07/Jan/19 Resolved: 21/Sep/18 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.11.0, Lustre 2.12.0, Lustre 2.10.4 |
| Fix Version/s: | Lustre 2.12.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Sebastien Piechurski | Assignee: | Nathaniel Clark |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Environment: |
RHEL 7.5, dkms |
||
| Issue Links: |
|
||||
| Severity: | 3 | ||||
| Rank (Obsolete): | 9223372036854775807 | ||||
| Description |
|
While experimenting with the dkms package for lustre to implement a patchless server, I saw that the lustre-ldiskfs-dkms package does not provide the ldiskfs.ko module. It does provide the osd-ldiskfs module however. When playing the "dkms build" command, the logs show that the module is actually generated, but not copied in the dkms tree after that. It would seem the following lines are missing to the dkms.conf file:
--- dkms.conf.orig 2018-07-09 16:06:31.257592111 +0200
+++ dkms.conf 2018-07-06 05:20:44.913000000 +0200
@@ -107,3 +107,6 @@
BUILT_MODULE_NAME[${#BUILT_MODULE_NAME[@]}]="osd_ldiskfs"
BUILT_MODULE_LOCATION[${#BUILT_MODULE_LOCATION[@]}]="lustre/osd-ldiskfs/"
DEST_MODULE_LOCATION[${#DEST_MODULE_LOCATION[@]}]="/extra/lustre/"
+BUILT_MODULE_NAME[${#BUILT_MODULE_NAME[@]}]="ldiskfs"
+BUILT_MODULE_LOCATION[${#BUILT_MODULE_LOCATION[@]}]="ldiskfs/"
+DEST_MODULE_LOCATION[${#DEST_MODULE_LOCATION[@]}]="/extra/lustre/"
|
| Comments |
| Comment by Gerrit Updater [ 09/Jul/18 ] |
|
Sebastien Piechurski (sebastien.piechurski@atos.net) uploaded a new patch: https://review.whamcloud.com/32800 |
| Comment by Peter Jones [ 09/Jul/18 ] |
|
Nathaniel Can you please evaluate this proposed change? Thanks Peter |
| Comment by Gerrit Updater [ 09/Jul/18 ] |
|
Nathaniel Clark (nclark@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/32801 |
| Comment by Gerrit Updater [ 21/Sep/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32800/ |
| Comment by Peter Jones [ 21/Sep/18 ] |
|
Landed for 2.12 |