[LU-13308] update changelog_ext_nid to handle IPv6 addresses

• Type: Improvement
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.16.0, Lustre 2.15.6
• Labels: IPv6
• Assignee: Neil Brown
• Reporter: Andreas Dilger

Description

In struct changelog_ext_nid there is the possibility of using a larger lnet_nidv6_t to store IPv6 NIDs. This needs to be implemented in print_changelog_rec(), lfs_changelog() to understand the larger NIDs. It seems possible to just store the larger NID therein, and check whether extra and padding are zero to distinguish between IPv4 and IPv6 addresses. It would probably make sense to use the first two __u64 for the IPv6 address, and padding for the lnet network type/number.

struct changelog_ext_nid {
        /* have __u64 instead of lnet_nid_t type for use by client api */
        __u64 cr_nid;
        /* for use when IPv6 support is added */
        __u64 extra;    
        __u32 padding;  
};

Issue Links

• is related to: LU-16722 MGS config log restructuring
• is related to: LU-10391 LNET: Support IPv6
• is related to: LU-9727 Lustre Audit with Changelogs
• is related to: LU-15373 changelog improvements tracking

Comments

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/54982/
Subject: LU-13308 mdc: support additional flags for OBD_IOC_CHLG_POLL ioctl
Project: fs/lustre-release
Branch: b2_15
Current Patch Set:
Commit: 487e1c79e4c6570d91740f23106cc3ddfaa20236

[Peter Jones]

Merged for 2.16

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50099/
Subject: LU-13308 mdd: add LLOG_F_EXT_X_NID_BE
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: ad5d6d47dcfcdad720ec3e73cd5be2d5afe83cb1

[Gerrit Updater]

"James Simmons <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/54982
Subject: LU-13308 mdc: support additional flags for OBD_IOC_CHLG_POLL ioctl
Project: fs/lustre-release
Branch: b2_15
Current Patch Set: 1
Commit: 239dddfb066e0e66d4e905f461498a8b84613a3a

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/52361/
Subject: LU-13308 mdc: support additional flags for OBD_IOC_CHLG_POLL ioctl
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8320394725180b76e76f36b8a513f3c7bf11e65c

[James A Simmons]

I tracked down the reason for the test failiures. The function mdd_create() has it clf_flags ALWAYS set to zero so CLFE_NID_BE is never set. Maybe spec->sp_cr_flags might have the proper flags. If not does anyone know a way to extract the open flags we find for mdd_objects?

[Gerrit Updater]

"James Simmons <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/52361
Subject: LU-13308 mdc: support additional flags for OBD_IOC_CHLG_POLL ioctl
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b36c914431d357061a41edc105a02962db859a9e

[James A Simmons]

https://review.whamcloud.com/#/c/fs/lustre-release/+/50099