diff options
author | Philipp Reisner <philipp.reisner@linbit.com> | 2012-08-28 05:07:56 -0400 |
---|---|---|
committer | Philipp Reisner <philipp.reisner@linbit.com> | 2012-11-09 08:08:23 -0500 |
commit | 07fc96197aec46f7ad4f08a4b2a1ea426112e24d (patch) | |
tree | b1cb7a0f46bf938cd994515e91ed814f5f3ef976 | |
parent | 892fdd1aee35803bfa41ccf4df850ddb70f0811c (diff) |
drbd: Do not check aspects that are not subject to change in _conn_requests_state()
When _conn_requests_state() is used to change other parts of the state
than the connection, do not check for a valid connection transition.
Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com>
Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>
-rw-r--r-- | drivers/block/drbd/drbd_state.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/drivers/block/drbd/drbd_state.c b/drivers/block/drbd/drbd_state.c index f3e6090d462b..9ae40c96c1be 100644 --- a/drivers/block/drbd/drbd_state.c +++ b/drivers/block/drbd/drbd_state.c | |||
@@ -1742,9 +1742,11 @@ _conn_request_state(struct drbd_tconn *tconn, union drbd_state mask, union drbd_ | |||
1742 | union drbd_state ns_max, ns_min, os; | 1742 | union drbd_state ns_max, ns_min, os; |
1743 | bool have_mutex = false; | 1743 | bool have_mutex = false; |
1744 | 1744 | ||
1745 | rv = is_valid_conn_transition(oc, val.conn); | 1745 | if (mask.conn) { |
1746 | if (rv < SS_SUCCESS) | 1746 | rv = is_valid_conn_transition(oc, val.conn); |
1747 | goto abort; | 1747 | if (rv < SS_SUCCESS) |
1748 | goto abort; | ||
1749 | } | ||
1748 | 1750 | ||
1749 | rv = conn_is_valid_transition(tconn, mask, val, flags); | 1751 | rv = conn_is_valid_transition(tconn, mask, val, flags); |
1750 | if (rv < SS_SUCCESS) | 1752 | if (rv < SS_SUCCESS) |