[LU-15811] simplify lower/upper AIO/DIO code

• Type: Bug
• Resolution: Fixed
• Priority: Major
• Fix Version/s: Lustre 2.16.0, Lustre 2.15.2
• Severity: 3
• Assignee: Patrick Farrell
• Reporter: Patrick Farrell

Description

One of the patches for LU-13799, "Implement lower/upper aio" (https://review.whamcloud.com/44209/) created a complicated setup where the cl_dio_aio struct was used both for the top level DIO or AIO and for the lower level sub I/Os (corresponding to stripes).

This is quite complicated and hard to follow, so this rewrites these two uses to be separate structs.  This incidentally fixes at least one possible memory leak.

This is an attempt to clean up the code to make some bugs easier to identify.  This may be a prelude to moving the top level AIO to a different struct from the DIO.

Issue Links

• is related to: LU-16132 NULL pointer dereference lu_object_put

Comments

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/48456/
Subject: LU-15811 llite: Refactor DIO/AIO free code
Project: fs/lustre-release
Branch: b2_15
Current Patch Set:
Commit: 076938e18315d05ae8da49270a7c43876fccfadf

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/48455/
Subject: LU-15811 llite: Unify range unlock
Project: fs/lustre-release
Branch: b2_15
Current Patch Set:
Commit: 505c13b066547f41c80c90227be20259c0d22f35

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/48454/
Subject: LU-15811 llite: Rework upper/lower DIO/AIO
Project: fs/lustre-release
Branch: b2_15
Current Patch Set:
Commit: 1aca3426cde3ca3457bb3fa44c70ac91922df13d

[Peter Jones]

Landed for 2.16

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/48115/
Subject: LU-15811 llite: Refactor DIO/AIO free code
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f1c8ac1156ebea2b8e94a75f056c403973bb9010

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/48000/
Subject: LU-15811 llite: Unify range unlock
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 36c34af60767bd5da5e9a66df0297e1dc7fa1fb1

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/47187/
Subject: LU-15811 llite: Rework upper/lower DIO/AIO
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 51c18539338f1a23f3fa725470b5d36941b15c46

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/48115
Subject: LU-15811 llite: Refactor DIO/AIO free code
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 7d83fa1abc4c23308b42eec381f1da450ff7c8ce

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/48000
Subject: LU-15811 llite: Unify range unlock
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f197d66fe509b891a17f5ccf7af1f239a3eb296b

[Gerrit Updater]

"Patrick Farrell <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/47187
Subject: LU-15811 llite: Rework upper/lower DIO/AIO
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: fb8b38b8f6e91889923b60c543c23c6da9cad9f6