[LU-10346] utils: mkfs.lustre compiled from the git repositery references a missing lib Created: 07/Dec/17 Updated: 05/Aug/20 Resolved: 18/Dec/17 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.11.0, Lustre 2.10.3 |
| Type: | Bug | Priority: | Minor |
| Reporter: | CEA | Assignee: | Quentin Bouget |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | patch | ||
| Issue Links: |
|
||||
| Severity: | 3 | ||||
| Rank (Obsolete): | 9223372036854775807 | ||||
| Description |
|
Trying to run llmount.sh without first running 'make install' fails with EINVAL at the formatting step. The faulty code is: struct module_backfs_ops *load_backfs_module(enum ldd_mount_type mount_type)
{
(...)
/* Check for $LUSTRE environment variable from test-framework.
* This allows using locally built modules to be used.
*/
if (handle == NULL) {
char *dirname;
dirname = getenv("LUSTRE");
if (dirname) {
snprintf(filename, sizeof(filename),
>>>> "%s/utils/.libs/mount_%s.so",
dirname, fsname);
handle = dlopen(filename, RTLD_LAZY);
}
}
(...)
}
|
| Comments |
| Comment by James A Simmons [ 07/Dec/17 ] |
|
Also I see another issue as well. If you build with ./configure --disable-shared this those plugins will be built into mount.lustre directly. You can't load the plugins at all in that case. |
| Comment by James A Simmons [ 07/Dec/17 ] |
|
Oh I looked again and see PLUGIN is used to guard against loading plugins with static libraries only. So its a simple fix. |
| Comment by Quentin Bouget [ 07/Dec/17 ] |
|
Quentin Bouget (quentin.bouget@cea.fr) uploaded a new patch: https://review.whamcloud.com/30431 |
| Comment by Gerrit Updater [ 17/Dec/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30431/ |
| Comment by Minh Diep [ 18/Dec/17 ] |
|
Landed for 2.11 |
| Comment by Gerrit Updater [ 18/Dec/17 ] |
|
Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/30584 |
| Comment by Gerrit Updater [ 04/Jan/18 ] |
|
John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/30584/ |