Bug 607345
Summary: | Review comments on Storage Administration Guide - I/O Limits | ||
---|---|---|---|
Product: | Red Hat Enterprise Linux 6 | Reporter: | Tom Coughlan <coughlan> |
Component: | doc-Storage_Admin_Guide | Assignee: | Don Domingo <ddomingo> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | 6.0 | CC: | mdoyle, msnitzer |
Target Milestone: | rc | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-06-30 22:44:33 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 561593 |
Description
Tom Coughlan
2010-06-23 21:02:42 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red Hat Enterprise Linux major release. Product Management has requested further review of this request by Red Hat Engineering, for potential inclusion in a Red Hat Enterprise Linux Major release. This request is not yet committed for inclusion. thanks Mike, Tom. I've updated the doc: http://brisvegas.bne.redhat.com/rhel6-storageguide/index-single.html#newstorage-iolimits please let me know if any further revisions are required. (In reply to comment #2) > thanks Mike, Tom. I've updated the doc: > > http://brisvegas.bne.redhat.com/rhel6-storageguide/index-single.html#newstorage-iolimits > > please let me know if any further revisions are required. I'm _really_ not a fan of s/limits/parameters/. I think we seized on NEC's concern about "limits" way too aggressively but I'll accept it (as I'm quite biased being so close to these details.. I also worked harder than I'd care to have done to switch peoples' thinking from "topology" to "limits"!). 'limits' wasn't something out of left field. Case in point the BLOCK LIMITS VPD page provides alignment and {physical,logical}_block_size information. Also, the kernel's data structure where this info is stored is 'struct queue_limits'. But, whatever... "parameters" it is. Don, here are some Edit notes: - s/frmo/from/ in the opening paragraph. - in 19.6, we need to correct this: "If a device doesn't provide I/O parameters information, fdisk will align all partitions on a 1MB boundary." to be something like: "fdisk will align all partitions on a 1MB boundary." See this for more details: https://bugzilla.redhat.com/show_bug.cgi?id=179390#c60 - 19.6: "As such, by default all partitions will align on a 1MB boundary." should be: "As such, by default all partitions will be aligned on a 1MB boundary." thanks Mike. I've applied your revisions accordingly, and updated the test build. (In reply to comment #3) > 'limits' wasn't something out of left field. Case in point the BLOCK LIMITS > VPD page provides alignment and {physical,logical}_block_size information. From SBC-3: 6.5.3 Block Limits VPD page The Block Limits VPD page provides the application client with the means to obtain certain operating parameters of the logical unit. ---------- :) (In reply to comment #5) > (In reply to comment #3) > > > 'limits' wasn't something out of left field. Case in point the BLOCK LIMITS > > VPD page provides alignment and {physical,logical}_block_size information. > > From SBC-3: > > 6.5.3 Block Limits VPD page > The Block Limits VPD page provides the application client with the means to > obtain certain operating parameters of the logical unit. > ---------- > > :) Well played sir... |