[LU-11783] lustre/lustre_user.h is deprecated too soon

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.12.0
• Fix Version/s: Lustre 2.12.0, Lustre 2.10.7
• Type: Bug
• Priority: Minor
• Reporter: Andreas Dilger
• Assignee: Andreas Dilger
• Resolution: Fixed
• Labels: easy
• Severity: 3

Description

The "lustre/lustre_user.h" header has been in use for many years. The patch https://review.whamcloud.com/25246 "LU-6401 uapi: migrate remaining uapi headers to uapi directory" moved the header to "linux/lustre/lustre_user.h" and left a stub "lustre/lustre_user.h" behind that generates a compiler warning that this header is deprecated.

However, no window was given between the introduction of the new header and the deprecation of the old header, which makes it difficult for applications to smoothly transition to the new header location. Also, installing Lustre headers into the "linux/" directory before Lustre is actually part of the kernel may potentially cause problems.

Disable the deprecation warning in the old header for several releases, until the new header location has been available for a good time and it is safe for applications to assume that it is available.

Comments

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33872
Subject: LU-11783 build: remove lustre_user.h deprecation warning
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4bb072344abf0f0bf786ef6f2463e0ed4f6acde9

[James A Simmons]

Yeah its gone it be many many years before the /usr/include/lustre will go away.

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33876
Subject: LU-11783 utils: fix warnings when lustre_user.h included
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 38f5e3f131fbc1cf131558340f74cd913a3c7b4b

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/33876/
Subject: LU-11783 utils: fix warnings when lustre_user.h included
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: db0592145574c5ad22a7b7372b06ba2da7d85a60

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/33872/
Subject: LU-11783 build: remove lustre_user.h deprecation warning
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 5b468e20ecc0b77460444393d1bb41867d99899d

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/34064
Subject: LU-11783 utils: fix warnings when lustre_user.h included
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: d57efdc8789625559f1222d4b8c9148fd5b53d0c

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/34064/
Subject: LU-11783 utils: fix warnings when lustre_user.h included
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: 3cf95c0bd388489c75382083a5b9a9b99d769415