[LU-14047] change EWOULDBLOCK to EAGAIN, add EWOULDBLOCK to spelling.txt Created: 20/Oct/20 Updated: 26/Feb/21 Resolved: 26/Feb/21 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.15.0 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | John Hammond | Assignee: | John Hammond |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | easy | ||
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
It's possible to find places in the clio stack where rc is set to -EWOULDBLOCK and later compared to -EAGAIN. And if we're looking at the logs and wonder where rc=-11 came from then we much search for both constants. This is confusing and makes analysis harder than it should be. We should change all EWOULDBLOCK instances to EAGAIN and add EWOULDBLOCK||EAGAIN to spelling.txt. fs/lustre-release$ git grep EAGAIN | wc -l
272
fs/lustre-release$ git grep EWOULDBLOCK | wc -l
35
|
| Comments |
| Comment by Gerrit Updater [ 20/Oct/20 ] |
|
John L. Hammond (jhammond@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/40307 |
| Comment by Gerrit Updater [ 26/Feb/21 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/40307/ |
| Comment by Peter Jones [ 26/Feb/21 ] |
|
Landed for 2.15 |