rbd: don't zero-fill non-image object requests
authorAlex Elder <elder@inktank.com>
Wed, 27 Mar 2013 14:16:30 +0000 (09:16 -0500)
committerSage Weil <sage@inktank.com>
Fri, 29 Mar 2013 18:32:07 +0000 (11:32 -0700)
commit6e2a4505dba0cae8faa701426185dfb7b49f537c
tree7421cf34fea522ab761138d48be05aec06f4f635
parentd6c0dd6b0c196979fa7b34c1d99432fcb1b7e1df
rbd: don't zero-fill non-image object requests

A result of ENOENT from a read request for an object that's part of
an rbd image indicates that there is a hole in that portion of the
image.  Similarly, a short read for such an object indicates that
the remainder of the read should be interpreted a full read with
zeros filling out the end of the request.

This behavior is not correct for objects that are not backing rbd
image data.  Currently rbd_img_obj_request_callback() assumes it
should be done for all objects.

Change rbd_img_obj_request_callback() so it only does this zeroing
for image objects.  Encapsulate that special handling in its own
function.  Add an assertion that the image object request is a bio
request, since we assume that (and we currently don't support any
other types).

This resolves a problem identified here:
    http://tracker.ceph.com/issues/4559

The regression was introduced by bf0d5f503dc11d6314c0503591d258d60ee9c944.

Reported-by: Dan van der Ster <dan@vanderster.com>
Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-off-by: Sage Weil <sage@inktank.com>
drivers/block/rbd.c