[LU-13528] sanity test_69: read succeeded, expect -ENOENT Created: 07/May/20 Updated: 27/May/20 Resolved: 27/May/20 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.14.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Maloo | Assignee: | Sebastien Buisson |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Severity: | 3 |
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
This issue was created by maloo for S Buisson <sbuisson@ddn.com> This issue relates to the following test suite run: https://testing.whamcloud.com/test_sets/2be4adce-b792-4432-8f8b-6caba6148d68 test_69 failed with the following error: read succeeded, expect -ENOENT This error occurs only on aarch64 platforms. It seems the fail_loc value set on server side has no effect. VVVVVVV DO NOT REMOVE LINES BELOW, Added by Maloo for auto-association VVVVVVV |
| Comments |
| Comment by Sebastien Buisson [ 07/May/20 ] |
|
After further investigation, it appears that a bug with the Lustre code on aarch64 platforms turns the O_DIRECT IO into a regular IO, hence bypassing the expected fail_loc behavior in sanity test_69. The bug stems from the fact that the MAX_DIO_SIZE value is truncated to its lower 32 bits. #define MAX_DIO_SIZE ((MAX_MALLOC / sizeof(struct brw_page) * PAGE_SIZE) & \ ~(DT_MAX_BRW_SIZE - 1)) On 4kB PAGE_SIZE platforms, it goes unnoticed as the value fits into 32 bits. But on 64kB PAGE_SIZE platforms, this value can go up to 1365GB. This problem, in combination with the fact that patch https://review.whamcloud.com/36144 increased the size of struct brw_page to 32 bytes, makes 1024GB (64 bits) truncated to 0 (lower 32 bits). The consequence is that ll_direct_IO_impl() triggers a direct IO of 0 bytes, which in turn makes llite issue a normal, buffered IO. |
| Comment by Gerrit Updater [ 07/May/20 ] |
|
Sebastien Buisson (sbuisson@ddn.com) uploaded a new patch: https://review.whamcloud.com/38526 |
| Comment by Gerrit Updater [ 27/May/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/38526/ |
| Comment by Peter Jones [ 27/May/20 ] |
|
Landed for 2.14 |