Merge lp://qastaging/~timg-tpi/ureadahead/LP491943 into lp://qastaging/ureadahead

Proposed by Tim Gardner
Status: Needs review
Proposed branch: lp://qastaging/~timg-tpi/ureadahead/LP491943
Merge into: lp://qastaging/ureadahead
Diff against target: 78 lines (+34/-2)
1 file modified
src/trace.c (+34/-2)
To merge this branch: bzr merge lp://qastaging/~timg-tpi/ureadahead/LP491943
Reviewer Review Type Date Requested Status
Scott James Remnant (Canonical) Pending
Review via email: mp+33251@code.qastaging.launchpad.net

Description of the change

+ * Count the number of CPUs and divide buffer_size_kb by the number of CPUs.
+ Users should watch for instances of "mmiotrace has lost events" in dmesg to make
+ sure the trace buffers are not too small. The original value for
+ buffer_size_kb was chosen somewhat arbitrarily. Empirical testing
+ has shown that its large enough, so we don't actually know where the lower
+ boundary lies.
+ -LP: #491943

To post a comment you must log in.

Unmerged revisions

115. By Tim Gardner

Count the number of CPUs

Count the number of CPUs and divide buffer_size_kb by the number of CPUs.
Users should watch for instances of "mmiotrace has lost events" in dmesg to make
sure the trace buffers are not too small. The original value for
buffer_size_kb was chosen somewhat arbitrarily. Empirical testing
has shown that its large enough, so we don't actually know where the lower
boundary lies.
-LP: #491943

114. By Tim Gardner

Restore buffer_size_kb, LP501715

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches