libceph: allocate the locator string with GFP_NOFAIL
authorIlya Dryomov <idryomov@gmail.com>
Wed, 23 May 2018 12:46:53 +0000 (14:46 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Mon, 4 Jun 2018 18:46:00 +0000 (20:46 +0200)
commita86f009f106cba322c608785e09c8b5be8ffe8bb
tree1e5ce868d254c9b1ce38c30833e17715bfcdb460
parentc843d13caefad9f2f182f38d6bfe492c9f00e086
libceph: allocate the locator string with GFP_NOFAIL

calc_target() isn't supposed to fail with anything but POOL_DNE, in
which case we report that the pool doesn't exist and fail the request
with -ENOENT.  Doing this for -ENOMEM is at the very least confusing
and also harmful -- as the preceding requests complete, a short-lived
locator string allocation is likely to succeed after a wait.

(We used to call ceph_object_locator_to_pg() for a pi lookup.  In
theory that could fail with -ENOENT, hence the "ret != -ENOENT" warning
being removed.)

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
include/linux/ceph/osdmap.h
net/ceph/osd_client.c
net/ceph/osdmap.c