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

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 1.8.7, Lustre 1.8.6
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.3
• Type: Bug
• Priority: Minor
• Reporter: Mark Hills
• Assignee: Hongchao Zhang
• Resolution: Fixed

Environment

Linux, Lustre b1_8

• Severity: 3
• Bugzilla ID: 24,536

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

Comments

[Peter Jones]

Mark

Could you please upload the patch into gerrit? There are details on how to do this here - http://wiki.whamcloud.com/display/PUB/Using+Gerrit . If this is not clear enough, please let me know.

Thanks

Peter

[Peter Jones]

Hongchao

Please can you take care of porting this patch to master

Thanks

Peter

[Hongchao Zhang]

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

[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

[Andreas Dilger]

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

[Hongchao Zhang]

the patch has been refreshed

[Hongchao Zhang]

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

[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

[Peter Jones]

Landed for 2.11

[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

[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