[LU-10467] use standard linux wait_event macros

• Type: Improvement
• Resolution: Fixed
• Priority: Major
• Fix Version/s: Lustre 2.14.0
• Assignee: James A Simmons
• Reporter: James A Simmons

Description

Simplify l_wait_info() using the linux wait_event infrastructure.

Improvements are:

   - new wait_event_idle* macros are now in include/linux/wait.h which

     Ack from peterz.

   - all waits are now TASK_IDLE or TASK_INTERRUPTIBLE and so don't

     affect the load average.  There is no need to choose whether load

     is appropriate or not in each case.

   - all l_wait_event() users are handled so l_wait_event() is

     removed.  The one case I had left out before uses

     wait_event_idle_exclusive() with and option of using

     wait_event_idle_exclusive_lifo() is that ever gets approved.

Issue Links

• is related to: LU-10824 ll_umount being improper use of cached mountpoint from mount time
• is related to: LU-1882 conf-sanity test_45: umount2: Device or resource busy
• is related to: LU-8841 Replace l_wait_info calls that use zeroed info parameters
• is related to: LU-12362 kernel warning 'do not call blocking ops when !TASK_RUNNING ' in ptlrpcd
• is related to: LU-12436 Memory allocation failure error dropped
• is related to: LU-12511 Prepare lustre for adoption into the linux kernel
• is related to: LU-6431 do not call blocking ops when !TASK_RUNNING
• is related to: LU-4423 Tracking of patches from upstream kernel to Lustre client

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/38405/
Subject: LU-10467 ptlrpc: change LONG_UNLINK to PTLRPC_REQ_LONG_UNLINK
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 7c549da4014a7c7f7f662876115db16f4b05d74c

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/38403/
Subject: LU-10467 llite: clean up pcc_layout_wait()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: c5a311781fc60c9a91483b737d3f59096949439e

[Gerrit Updater]

Neil Brown ([e-mail]) uploaded a new patch: https://review.whamcloud.com/38405
Subject: LU-10467 ptlrpc: change LONG_UNLINK to PTLRPC_LONG_REQ_UNLINK
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 397675ad40fbfdea1dbd7e4667e38b3358501a24

[Gerrit Updater]

Neil Brown ([e-mail]) uploaded a new patch: https://review.whamcloud.com/38403
Subject: LU-10467 llite: clean up pcc_layout_wait()
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 24412377207fa449847bf193707fc74c2de9e42c

[Peter Jones]

Amazing! Congrats

[James A Simmons]

This work is now complete!!!!

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/37729/
Subject: LU-10467 libcfs: remove l_wait_event() and related macros.
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 39287411e32aeeed342c7da8417975e97310cd4a

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/37696/
Subject: LU-10467 ptlrpc: convert waiting in ptlrpc_hr_main()
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 5891fbfad266623631e89d9f349722343592220d

[Gerrit Updater]

Neil Brown ([e-mail]) uploaded a new patch: https://review.whamcloud.com/37729
Subject: LU-10467 libcfs: remove l_wait_event() and related macros.
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 017a47e4eaf756ec54a431ee17aadf37cdc7b888

[James A Simmons]

Down to two patches - 

https://review.whamcloud.com/#/c/37696/

https://review.whamcloud.com/#/c/37473/

Plus we will need one cleanup patch.