diff options
author | Boaz Harrosh <bharrosh@panasas.com> | 2010-07-29 17:08:13 +0300 |
---|---|---|
committer | Boaz Harrosh <bharrosh@panasas.com> | 2010-08-04 13:17:57 +0300 |
commit | 6e31609b1dcd595d7e4676ce62323532b29e8999 (patch) | |
tree | 1d43b7e9affee41df43db3657b50058da5645732 /sound | |
parent | b2848349296f3428850eb34c3a52d586f48d4b04 (diff) | |
download | linux-stable-6e31609b1dcd595d7e4676ce62323532b29e8999.tar.gz linux-stable-6e31609b1dcd595d7e4676ce62323532b29e8999.tar.bz2 linux-stable-6e31609b1dcd595d7e4676ce62323532b29e8999.zip |
exofs: Remove useless optimization
We used to compact all used devices in an IO to the beginning
of the device array in an io_state. And keep a last device used
so in later loops we don't iterate on all device slots. This
does not prevent us from checking if slots are empty since in
reads we only read from a single mirror and jump to the next
mirror-set.
This optimization is marginal, and needlessly complicates the
code. Specially when we will later want to support raid/456
with same abstract code. So remove the distinction between
"dev" and "comp". Only "dev" is used both as the device used
and as the index (component) in the device array.
[Note that now the io_state->dev member is redundant but I
keep it because I might want to optimize by only IOing a
single group, though keeping a group_width*mirrors devices
in io_state, we now keep num-devices in each io_state]
Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
Diffstat (limited to 'sound')
0 files changed, 0 insertions, 0 deletions