[LU-16565] Remove ldlm flag ldlm_{is,set,clear} macros - Whamcloud Community JIRA

Details

Type: Improvement
Resolution: Unresolved
Priority: Minor
Fix Version/s: None
Affects Version/s: None
Labels:
None

Rank (Obsolete):
9223372036854775807

Description

We have macros like ldlm_{is,set,clear}_#FLAG_NAME# for all of the ldlm flags. I'm going to remove them and convert all usages to just using the flag bit names directly.

They make it much harder to find all uses of the actual lock flags and we are entirely inconsistent on whether we use the macros or not. And since we're always going to use the flag names for multi-flag uses, we can't convert 100% to macros anyway.

Attachments

Activity

[LU-16565] Remove ldlm flag ldlm_{is,set,clear} macros

Gerrit Updater added a comment - 13/Mar/25 5:06 PM

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/57543/
Subject: LU-16565 quota: Remove ldlm is,set,clear macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 62eac36b7c9390f2696b3b29a990416ccfe29785

Gerrit Updater added a comment - 13/Mar/25 5:06 PM "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/57543/ Subject: LU-16565 quota: Remove ldlm is,set,clear macros Project: fs/lustre-release Branch: master Current Patch Set: Commit: 62eac36b7c9390f2696b3b29a990416ccfe29785

Gerrit Updater added a comment - 13/Mar/25 5:06 PM

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/57542/
Subject: LU-16565 target: Remove ldlm is,set,clear macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 475a1ba0769c96c20024b62fc32e02064e534cec

Gerrit Updater added a comment - 13/Mar/25 5:06 PM "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/57542/ Subject: LU-16565 target: Remove ldlm is,set,clear macros Project: fs/lustre-release Branch: master Current Patch Set: Commit: 475a1ba0769c96c20024b62fc32e02064e534cec

James A Simmons added a comment - 06/Feb/25 2:38 AM

Yes I agree it would be better to use the atomic bit operations instead.

James A Simmons added a comment - 06/Feb/25 2:38 AM Yes I agree it would be better to use the atomic bit operations instead.

Tim Day added a comment - 20/Dec/24 4:59 AM

After refreshing this, I wonder if I should be using the atomic bit operations instead i.e. set_bit, clear_bit, and so on.

Tim Day added a comment - 20/Dec/24 4:59 AM After refreshing this, I wonder if I should be using the atomic bit operations instead i.e. set_bit, clear_bit, and so on.

Gerrit Updater added a comment - 20/Dec/24 4:44 AM

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57550
Subject: LU-16565 ldlm: [3] Remove ldlm is,set,clear macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: d7af7113c0ff5f2065b07ee1295d91fdf331c660

Gerrit Updater added a comment - 20/Dec/24 4:44 AM "Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57550 Subject: LU-16565 ldlm: [3] Remove ldlm is,set,clear macros Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: d7af7113c0ff5f2065b07ee1295d91fdf331c660

Gerrit Updater added a comment - 20/Dec/24 4:42 AM

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57549
Subject: LU-16565 ldlm: [2] Remove ldlm is,set,clear macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4e7d2fea389bfc7f925ac79a693b7f976c1a1f69

Gerrit Updater added a comment - 20/Dec/24 4:42 AM "Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57549 Subject: LU-16565 ldlm: [2] Remove ldlm is,set,clear macros Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 4e7d2fea389bfc7f925ac79a693b7f976c1a1f69

Gerrit Updater added a comment - 20/Dec/24 4:41 AM

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57548
Subject: LU-16565 ldlm: [1] Remove ldlm is,set,clear macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: d3699de9b2b46df3a4ab6fb9a4ae1066a62650ab

Gerrit Updater added a comment - 20/Dec/24 4:41 AM "Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57548 Subject: LU-16565 ldlm: [1] Remove ldlm is,set,clear macros Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: d3699de9b2b46df3a4ab6fb9a4ae1066a62650ab

Gerrit Updater added a comment - 20/Dec/24 4:40 AM

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57547
Subject: LU-16565 mdc: Remove ldlm is,set,clear macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 319750223098327e30d7b7e8bf25afb85bbc7b54

Gerrit Updater added a comment - 20/Dec/24 4:40 AM "Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57547 Subject: LU-16565 mdc: Remove ldlm is,set,clear macros Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 319750223098327e30d7b7e8bf25afb85bbc7b54

Gerrit Updater added a comment - 20/Dec/24 4:40 AM

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57546
Subject: LU-16565 llite: Remove ldlm is,set,clear macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: cc1f609e341f988ebc307f574768c347ec4af28d

Gerrit Updater added a comment - 20/Dec/24 4:40 AM "Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57546 Subject: LU-16565 llite: Remove ldlm is,set,clear macros Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: cc1f609e341f988ebc307f574768c347ec4af28d

Gerrit Updater added a comment - 20/Dec/24 4:39 AM

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57545
Subject: LU-16565 mdt: Remove ldlm is,set,clear macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 6f4081093a7efc417de8038b3bd61ad066254b17

Gerrit Updater added a comment - 20/Dec/24 4:39 AM "Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57545 Subject: LU-16565 mdt: Remove ldlm is,set,clear macros Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 6f4081093a7efc417de8038b3bd61ad066254b17

Gerrit Updater added a comment - 20/Dec/24 4:38 AM

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57544
Subject: LU-16565 osc: Remove ldlm is,set,clear macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4ef61372cd8d05bfb46364bd54702d51ac15e72c

Gerrit Updater added a comment - 20/Dec/24 4:38 AM "Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/57544 Subject: LU-16565 osc: Remove ldlm is,set,clear macros Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 4ef61372cd8d05bfb46364bd54702d51ac15e72c

People

Assignee:: Patrick Farrell

Reporter:: Patrick Farrell

Votes:: 0 Vote for this issue

Watchers:: 4 Start watching this issue

Dates

Created:: 16/Feb/23 10:49 PM

Updated:: 10/Apr/25 6:50 AM