[LU-9761] Add ldiskfs support to dkms for patchless kernel

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.10.0
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.4
• Type: Task
• Priority: Major
• Reporter: Nathaniel Clark
• Assignee: Nathaniel Clark
• Resolution: Fixed

Description

Add support for ldiskfs backed storage to dkms build.

This should work for patched or (when LU-20 completes) patchless kernels.

Proposed Solution
Break out ldiskfs and zfs as seperate dkms packages. I don't think there's a real use case to have both available on the same server. This solves many issues of how to structure dependencies especially for zfs build (and the build dependency issues inherent in dkms when rebuilding for a new kernel).

I don't see a simple way to ONLY build osd-zfs.ko or ONLY build osd-ldiskfs.ko so I didn't break the osd portions out, but the new dkms packages will be lustre-modules + lustre-osd-zfs or lustre-modules + lustre-osd-ldiskfs, or lustre-modules + both osd's.

Issue Links

• is blocking: LU-10676 lustre-zfs-dkms rpm should Provide lu...
• is blocked by: LU-20 patchless server kernel

Comments

[Gerrit Updater]

Nathaniel Clark ([e-mail]) uploaded a new patch: https://review.whamcloud.com/27990
Subject: LU-9761 dkms: Add ldiskfs dkms support
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f3128fee3e8843fa424784fba95488d806ab6e73

[Brian Murrell (Inactive)]

I don't think there's a real use case to have both available on the same server.

There is actually a very real case and that's when using IML. IML requires that all storage servers (a.k.a. Lustre servers) support both ldiskfs and zfs because IML needs to be able to scan the system for targets of both types and doesn't know ahead of time which it might find.

[Nathaniel Clark]

Sorry, I copied and pasted the description, I've been around on this with Chris, and the patch actually crates 3 separate server dkms options, ldiskfs, zfs and all.

[Oleg Drokin]

I am kind of surprised why do we need ldiskfs dkms when lots of people oppose having full build environments on every node. On zfs there's at least soem value for patched servers case because we cannot distribute binaries, but ldiskfs?

[Brian Murrell (Inactive)]

I would guess it's because some people would accept having full build environments on nodes in exchange for having a hands-off distro-kernel upgrading procedure.

[Oleg Drokin]

Patchless server gives you that upgrading procedure. For project quotas patched kernels there is always a manual kernel building step.

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/31201
Subject: LU-9761 dkms: Add ldiskfs dkms support
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: fd03941dee71753e0ec05e18fb221e74500a2186

[Gerrit Updater]

John L. Hammond ([e-mail]) merged in patch https://review.whamcloud.com/31201/
Subject: LU-9761 dkms: Add ldiskfs dkms support
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: e8757177715993bf38f732808af1957cc30f0fd1

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/27990/
Subject: LU-9761 dkms: Add ldiskfs dkms support
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 64a49a1d42b44dfdbc2f951177be3cbb9cd368f9

[Peter Jones]

Landed for 2.11