[LU-12610] Get rid of OBD_ macros that just redefine CFS_ macros Created: 29/Jul/19  Updated: 03/Nov/23  Resolved: 03/Nov/23

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Improvement Priority: Minor
Reporter: Ben Evans (Inactive) Assignee: Tim Day
Resolution: Fixed Votes: 0
Labels: easy, lug23dd

Issue Links:
Related
is related to LU-6142 Enforce Linux kernel coding style in ... Open
Rank (Obsolete): 9223372036854775807

 Description   

#define OBD_FAIL_CHECK(id) CFS_FAIL_CHECK(id)

 

Really?



 Comments   
Comment by James A Simmons [ 29/Jul/19 ]

Yes please!!

Comment by Gerrit Updater [ 29/Jul/19 ]

Ben Evans (bevans@cray.com) uploaded a new patch: https://review.whamcloud.com/35640
Subject: LU-12610 obd: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 52bdc94e2529a8314ef8f4cb209abd2bfab625cc

Comment by Gerrit Updater [ 18/Apr/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50665
Subject: LU-12610 contrib: spelling warning for OBD_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 50636a3fd60117713b1f0a1afece051442a404e4

Comment by Gerrit Updater [ 19/Apr/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50684
Subject: LU-12610 ptlrpc: replace OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7a07b128e469ba066e6feebece2a5e5c5b97b265

Comment by Gerrit Updater [ 19/Apr/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50685
Subject: LU-12610 ldlm: replace OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 55fb99a95580cb713d91f21c62816c5c49e21e02

Comment by Gerrit Updater [ 26/Apr/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50665/
Subject: LU-12610 contrib: spelling warning for OBD_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: a136e3321b266b6b7401f9dffd940204fe1195ad

Comment by Gerrit Updater [ 29/Apr/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50804
Subject: LU-12610 llite: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 90f5d5db3c7ec5e98390b504559165ddbc13085c

Comment by Gerrit Updater [ 29/Apr/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50805
Subject: LU-12610 osd: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 64a9ccc5e6137f7dadacd34cc7187a22a9aa25e5

Comment by Gerrit Updater [ 29/Apr/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50806
Subject: LU-12610 mdt: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 39d551635c0c39d6d30404b6b2ca2ce98fb4612a

Comment by Gerrit Updater [ 29/Apr/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50807
Subject: LU-12610 tests: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 295316955ae6976e9980f15dca0a7c31df822822

Comment by Gerrit Updater [ 29/Apr/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50808
Subject: LU-12610 obd: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 1692470a7238fa7a89124e349944a588c187dc33

Comment by Gerrit Updater [ 29/Apr/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50809
Subject: LU-12610 misc: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 74958e6f5f03a2ad42c02b64be026a9c70400b0c

Comment by Gerrit Updater [ 19/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50804/
Subject: LU-12610 llite: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 407d34a639c93dc3b4c984249a632cee70c25d05

Comment by Gerrit Updater [ 19/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50807/
Subject: LU-12610 tests: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8bb6793d7bc903ebb8bf48dff22364ef30151fc8

Comment by Gerrit Updater [ 19/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50808/
Subject: LU-12610 obd: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 24ada119f28045eba84f6089ca8761165d7efbec

Comment by Gerrit Updater [ 24/May/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51123
Subject: LU-12610 target: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 117b104deae4ab49a8608db47eeae33c44cfff3e

Comment by Gerrit Updater [ 24/May/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51124
Subject: LU-12610 osc: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b8adcca0b4b7902d934303b46653cd40131d0a95

Comment by Gerrit Updater [ 31/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50685/
Subject: LU-12610 ldlm: replace OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 5770020076b339f82d8663f53a4312cba47b08d9

Comment by Gerrit Updater [ 31/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50806/
Subject: LU-12610 mdt: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 32aff098ce016fb56d0414625236c26d88de3956

Comment by Gerrit Updater [ 09/Jun/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50684/
Subject: LU-12610 ptlrpc: replace OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 3e24d50046e59a55274cba29197236f570796a3d

Comment by Gerrit Updater [ 09/Jun/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50805/
Subject: LU-12610 osd: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8a7ce3cf279d8742edf219dd96c48c7882a57f4e

Comment by Gerrit Updater [ 09/Jun/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50809/
Subject: LU-12610 misc: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 424fa1bff070d0382c3ad5073361972088331857

Comment by Gerrit Updater [ 09/Jun/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/51123/
Subject: LU-12610 target: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: d9612729dd8d96fad7d783519cee47c1ca543286

Comment by Gerrit Updater [ 09/Jun/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/51124/
Subject: LU-12610 osc: remove OBD_ -> CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: ef01e035b3307e20e25ad623b8e2a955462a5a32

Comment by Gerrit Updater [ 13/Jun/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51291
Subject: LU-12610 cfs: add unlikely to CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b6a1968127141a1d2057f11e33bad0f1f35de58b

Comment by Gerrit Updater [ 20/Jun/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/51291/
Subject: LU-12610 cfs: add unlikely to CFS_ macros
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: bbfe5c8d211c8285a7d426f3efc92b13028e638d

Comment by Gerrit Updater [ 03/Nov/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/35640/
Subject: LU-12610 obd: delete OBD_ -> CFS_ redefinitions
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: b4ec4dd10a7d0109e931e0490daddd98105457c3

Comment by Peter Jones [ 03/Nov/23 ]

there are no open patches tracked under this ticket still to land. Are there more to come or can this ticket be marked as resolved?

Comment by Tim Day [ 03/Nov/23 ]

This should be fine to resolve.

Generated at Sat Feb 10 02:54:06 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.