Conversation
doc/requirements/io_class
Outdated
| --- | ||
|
|
||
| The product shall allow to pin an IO class. Data from the pinned IO class shall | ||
| not be evicted unless its occupancy limit is not exceeded. |
There was a problem hiding this comment.
This means the opposite of what I think you intended. It should be "unless its occupancy limit is exceeded".
There was a problem hiding this comment.
Data from -> cachelines assigned to?
There was a problem hiding this comment.
I think stating that something should not happen in an unspecified set of circumstances makes it hard to test. Instead I would specify a set of requirements exactly describing expected eviction/replacement behavior. This would also explain the interdependencies between specific settings (pinning, eviction priority, max occupancy) which might be hard to deduce from the existing form. Let's discuss this.
There was a problem hiding this comment.
How exactly would you like to extend this req?
Signed-off-by: Michal Mielewczyk <michal.mielewczyk@intel.com>
4 participants
Signed-off-by: Michal Mielewczyk michal.mielewczyk@intel.com