nagai/kernel
3
0
Fork 0
forked from mirrors/linux
Code Pull requests Activity

libceph: behave in mon_fault() if cur_mon < 0

Browse source 
This can happen if __close_session() in ceph_monc_stop() races with
a connection reset.  We need to ignore such faults, otherwise it's
likely we would take !hunting, call __schedule_delayed() and end up
with delayed_work() executing on invalid memory, among other things.

The (two!) con->private tests are useless, as nothing ever clears
con->private.  Nuke them.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
This commit is contained in:
Ilya Dryomov 2016-01-23 15:57:51 +01:00
parent bee3a37c47
commit b5d91704f5
1 changed files with 9 additions and 14 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

23
net/ceph/mon_client.c
View file

@ -1155,22 +1155,17 @@ static void mon_fault(struct ceph_connection *con)
{ {
struct ceph_mon_client *monc = con->private; struct ceph_mon_client *monc = con->private;
if (!monc)
return;
dout("mon_fault\n");
mutex_lock(&monc->mutex); mutex_lock(&monc->mutex);
if (!con->private) dout("%s mon%d\n", __func__, monc->cur_mon);
goto out; if (monc->cur_mon >= 0) {
if (!monc->hunting) {
if (!monc->hunting) { dout("%s hunting for new mon\n", __func__);
dout("%s hunting for new mon\n", __func__); reopen_session(monc);
reopen_session(monc); __schedule_delayed(monc);
__schedule_delayed(monc); } else {
} else { dout("%s already hunting\n", __func__);
dout("%s already hunting\n", __func__); }
} }
out:
mutex_unlock(&monc->mutex); mutex_unlock(&monc->mutex);
} }