diff options
author | Boaz Harrosh <bharrosh@panasas.com> | 2012-08-02 15:38:23 +0300 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-08-02 17:42:51 -0400 |
commit | 7de6e28417c65919cf2c1621841a650c4a3afbbd (patch) | |
tree | 2dce28c9c8163082ce111e42ff902879ddb8a53c /fs/fs-writeback.c | |
parent | f6166384095b7ecf77752b5e9096e6d03d75f7ae (diff) | |
download | talos-op-linux-7de6e28417c65919cf2c1621841a650c4a3afbbd.tar.gz talos-op-linux-7de6e28417c65919cf2c1621841a650c4a3afbbd.zip |
pnfs-obj: Better IO pattern in case of unaligned offset
Depending on layout and ARCH, ORE has some limits on max IO sizes
which is communicated on (what else) ore_layout->max_io_length,
which is always stripe aligned.
This was considered as the pg_test boundary for splitting and starting
a new IO.
But in the case of a long IO where the start offset is not aligned
what would happen is that both end of IO[N] and start of IO[N+1]
would be unaligned, causing each IO boundary parity unit to be
calculated and written twice.
So what we do in this patch is split the very start of an unaligned
IO, up to a stripe boundary, and then next IO's can continue fully
aligned til the end.
We might be sacrificing the case where the full unaligned IO would
fit within a single max_io_length, but the sacrifice is well worth
the elimination of double calculation and parity units IO.
Actually the sacrificing is marginal and is almost unmeasurable.
TODO:
If we know the total expected linear segment that will
be received, at pg_init, we could use that information
in many places:
1. blocks-layout get_layout write segment size
2. Better mds-threshold
3. In above situation for a better clean split
I will do this in future submission.
Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/fs-writeback.c')
0 files changed, 0 insertions, 0 deletions