Commit 2140ac26 authored by Karsten Graul's avatar Karsten Graul Committed by David S. Miller
Browse files

net/smc: simplify link deactivation



Cancel the testlink worker during link clear processing and remove the
extra function smc_llc_link_inactive().

Signed-off-by: default avatarKarsten Graul <kgraul@linux.ibm.com>
Reviewed-by: default avatarUrsula Braun <ubraun@linux.ibm.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 1020e1ef
Loading
Loading
Loading
Loading
+2 −6
Original line number Diff line number Diff line
@@ -261,7 +261,7 @@ static void smc_lgr_free_work(struct work_struct *work)
			struct smc_link *lnk = &lgr->lnk[i];

			if (smc_link_usable(lnk))
				smc_llc_link_inactive(lnk);
				lnk->state = SMC_LNK_INACTIVE;
		}
	}
	smc_lgr_free(lgr);
@@ -692,7 +692,7 @@ static void smc_lgr_cleanup(struct smc_link_group *lgr)
			struct smc_link *lnk = &lgr->lnk[i];

			if (smc_link_usable(lnk))
				smc_llc_link_inactive(lnk);
				lnk->state = SMC_LNK_INACTIVE;
		}
	}
}
@@ -706,16 +706,12 @@ static void __smc_lgr_terminate(struct smc_link_group *lgr, bool soft)
	struct smc_connection *conn;
	struct smc_sock *smc;
	struct rb_node *node;
	int i;

	if (lgr->terminating)
		return;	/* lgr already terminating */
	if (!soft)
		cancel_delayed_work_sync(&lgr->free_work);
	lgr->terminating = 1;
	if (!lgr->is_smcd)
		for (i = 0; i < SMC_LINKS_PER_LGR_MAX; i++)
			smc_llc_link_inactive(&lgr->lnk[i]);

	/* kill remaining link group connections */
	read_lock_bh(&lgr->conns_lock);
+4 −11
Original line number Diff line number Diff line
@@ -660,22 +660,15 @@ void smc_llc_link_deleting(struct smc_link *link)
	smc_wr_wakeup_tx_wait(link);
}

/* called in tasklet context */
void smc_llc_link_inactive(struct smc_link *link)
{
	if (link->state == SMC_LNK_INACTIVE)
		return;
	link->state = SMC_LNK_INACTIVE;
	cancel_delayed_work_sync(&link->llc_testlink_wrk);
	smc_wr_wakeup_reg_wait(link);
	smc_wr_wakeup_tx_wait(link);
}

/* called in worker context */
void smc_llc_link_clear(struct smc_link *link)
{
	flush_workqueue(link->llc_wq);
	destroy_workqueue(link->llc_wq);
	complete(&link->llc_testlink_resp);
	cancel_delayed_work_sync(&link->llc_testlink_wrk);
	smc_wr_wakeup_reg_wait(link);
	smc_wr_wakeup_tx_wait(link);
}

/* register a new rtoken at the remote peer */
+0 −1
Original line number Diff line number Diff line
@@ -56,7 +56,6 @@ int smc_llc_send_delete_link(struct smc_link *link,
int smc_llc_link_init(struct smc_link *link);
void smc_llc_link_active(struct smc_link *link, int testlink_time);
void smc_llc_link_deleting(struct smc_link *link);
void smc_llc_link_inactive(struct smc_link *link);
void smc_llc_link_clear(struct smc_link *link);
int smc_llc_do_confirm_rkey(struct smc_link *link,
			    struct smc_buf_desc *rmb_desc);