[LU-618] Reads are not included in IO accounting

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.3
• Affects Version/s: Lustre 1.8.7, Lustre 1.8.6
• Environment: Linux, Lustre b1_8
• Severity: 3
• Bugzilla ID: 24,536
• Assignee: Hongchao Zhang
• Reporter: Mark Hills

Description

Writes are accounted for, but not reads. The supplied patch fixes this.

It seems that the accounting it normally done in the kernels page writeback and readahead
functionality. Therefore as Lustre implements its own readahead, it must also maintain its own
accounting on reads (but not writes).

Previously discussed in:

• http://lists.lustre.org/pipermail/lustre-devel-lustre.org/2011-April/001586.html
• https://bugzilla.lustre.org/show_bug.cgi?id=24536

Issue Links

• Trackbacks: Lustre 1.8.x known issues tracker While testing against Lustre b18 branch, we would hit known bugs which were already reported in Lustre Bugzilla https://bugzilla.lustre.org/. In order to move away from relying on Bugzilla, we would create a JIRA

Comments

[Gerrit Updater]

John L. Hammond ([e-mail]) merged in patch https://review.whamcloud.com/30570/
Subject: LU-618 llite: IO accounting of page read
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: 19f21a32e74fd347f03c90b33b3d75ee537bf0e7

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30570
Subject: LU-618 llite: IO accounting of page read
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: 6f8fb1839bafac7a52dee5779c065fad337d526f

[Peter Jones]

Landed for 2.11

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/1636/
Subject: LU-618 llite: IO accounting of page read
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 48de9b195d71cb57ee50636d75c59d78ee068cee

[Hongchao Zhang]

the patch https://review.whamcloud.com/#/c/1636 has been updated against the latest master.

[Hongchao Zhang]

the patch has been refreshed

[Andreas Dilger]

This conflict with RHEL5 is probably gone, should the patch be refreshed?

[Hongchao Zhang]

there is a bug in OFED(1.5.3.2) which causes the patch can't be builded successfully in RHEL5, the detailed info is,

for RHEL5, there is no "task_io_account_cancelled_write" definition prior to 2.6.18-164.2.1 and is okay for OFED, which
defines the same function in the backport (in kernel_addons/backport directory), but the function is defined from
2.6.18-164.2.1, then cause the redefinition error when building Lustre with OFED

[Hongchao Zhang]

its patch is tracked at https://review.whamcloud.com/#/c/1636

[Peter Jones]

Hongchao

Please can you take care of porting this patch to master

Thanks

Peter