[LU-11825] Remove LU-8964/pio feature & supporting framework

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.13.0, Lustre 2.12.1
• Severity: 3
• Assignee: Patrick Farrell (Inactive)
• Reporter: Patrick Farrell (Inactive)

Description

As discussed recently on the lustre-devel mailing list:

http://lists.lustre.org/pipermail/lustre-devel-lustre.org/2018-November/008230.html

http://lists.lustre.org/pipermail/lustre-devel-lustre.org/2018-November/008231.html

http://lists.lustre.org/pipermail/lustre-devel-lustre.org/2018-November/008233.html

http://lists.lustre.org/pipermail/lustre-devel-lustre.org/2018-November/008234.html

I think the pio feature should be removed.

Quoting my email to the list:
"

elieve that long term it’s headed out of Lustre. It only improves performance in a limited way in certain circumstances, and harms it in various others. So it’s off by default, and, I suspect, remains completely unused. A while back I noticed its test framework test didn’t activate it correctly, and once fixed, it sometimes deadlocks (race with truncate). There’s a patch to fix that, but a problem was found in it and it has since languished.

I would still suggest you take it, Neil, as othewise you’ll complicate a bunch of potentially nasty porting working in the CLIO stack, as you apply the years of patches written with it there. Instead, I’d suggest we pull it in the open sfs branch (Sorry! It was a promising idea but it hasn’t panned out, and the current parallel readahead work isn’t going to use it.) and then eventually you could pick that up."

Patches forthcoming.  I have also asked the lustre-discuss mailing list if anyone is using it, and we can change course if someone pipes up.

Issue Links

• is related to: LU-12503 LustreError: 19435:0:(vvp_io.c:1056:vvp_io_write_start()) LBUG
• is related to: LU-10367 FIO Fails to run with libaio
• is related to: LU-10923 cl_io_loop improperly assumes all ios are rw-type IOs
• is related to: LU-12262 Improve sbi_flags checking
• is related to: LU-8964 use parallel I/O to improve performance on machines with slow single thread performance

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/34462/
Subject: LU-11825 llite: Remove ptask framework
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: 4e737a6a8a0f75425255c21eb95e43d9a950193b

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/34460/
Subject: LU-11825 clio: Remove pio feature
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: 81fe463c5ac1e8d5b51b680c1ddcd490bfc3e623

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/34462
Subject: LU-11825 llite: Remove ptask framework
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 536f10ee39ceeb705388b76f95ab08037ce54aac

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/34460
Subject: LU-11825 clio: Remove pio feature
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 8d3e6acbd3010d691a7982ef296068067925df51

[Peter Jones]

Landed for 2.13

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/33913/
Subject: LU-11825 llite: Remove ptask framework
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 2b045f3bc1729d43a8d9613d5609b838ad79ccc7

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/33912/
Subject: LU-11825 clio: Remove pio feature
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 2b0a34fe43bf4fce5560af61a45e5393c96070a9

[Gerrit Updater]

Patrick Farrell ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33913
Subject: LU-11825 llite: Remove ptask framework
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 49a31daa9aef492c08fac710bd579286ef0ec9fb

[Gerrit Updater]

Patrick Farrell ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33912
Subject: LU-11825 clio: Remove pio feature
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 6536783376edb5c69bd989098e2f4b34210583b1