diff options
| author | Guangliang Zhao <lucienchao@gmail.com> | 2014-04-01 16:22:15 +0200 |
|---|---|---|
| committer | Ilya Dryomov <idryomov@redhat.com> | 2014-10-14 19:03:29 +0200 |
| commit | c622d226155b12276ae3d29d546f4b314d7cd68c (patch) | |
| tree | ce4463c5d7f0b6f53237c184b4d18db3f55826ef | |
| parent | rbd: add img_obj_request_simple() helper (diff) | |
| download | linux-c622d226155b12276ae3d29d546f4b314d7cd68c.tar.xz linux-c622d226155b12276ae3d29d546f4b314d7cd68c.zip | |
rbd: skip the copyup when an entire object writing
It need to copyup the parent's content when layered writing,
but an entire object write would overwrite it, so skip it.
Signed-off-by: Guangliang Zhao <lucienchao@gmail.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Reviewed-by: Alex Elder <elder@linaro.org>
| -rw-r--r-- | drivers/block/rbd.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c index 6dae6586a8a9..16eb247cb5fb 100644 --- a/drivers/block/rbd.c +++ b/drivers/block/rbd.c @@ -2770,6 +2770,14 @@ static bool img_obj_request_simple(struct rbd_obj_request *obj_request) return true; /* + * Entire-object layered writes - we will overwrite whatever + * parent data there is anyway. + */ + if (!obj_request->offset && + obj_request->length == rbd_obj_bytes(&rbd_dev->header)) + return true; + + /* * If the object is known to already exist, its parent data has * already been copied. */ |