Commit 22f6c237 authored by Gil Fine's avatar Gil Fine Committed by Wen Zhiwei
Browse files

thunderbolt: Improve DisplayPort tunnel setup process to be more robust 

stable inclusion
from stable-v6.6.54
commit 4d0261cea469855fa3a33df8dd1cb29b7e6ee05a
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/IAZ3K2

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=4d0261cea469855fa3a33df8dd1cb29b7e6ee05a



--------------------------------

[ Upstream commit b4734507ac55cc7ea1380e20e83f60fcd7031955 ]

After DisplayPort tunnel setup, we add verification that the DPRX
capabilities read process completed. Otherwise, we bail out, teardown
the tunnel, and try setup another DisplayPort tunnel using next
available DP IN adapter. We do so till all DP IN adapters tried. This
way, we avoid allocating DP IN adapter and (bandwidth for it) for
unusable tunnel.

Signed-off-by: default avatarGil Fine <gil.fine@linux.intel.com>
Signed-off-by: default avatarMika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: default avatarQin Wan <qin.wan@hp.com>
Signed-off-by: default avatarAlexandru Gagniuc <alexandru.gagniuc@hp.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarWen Zhiwei <wenzhiwei@kylinos.cn>
parent 83a30f3d

drivers/thunderbolt/tb.c

+43 −41
Original line number Diff line number Diff line
@@ -1699,48 +1699,14 @@ static struct tb_port *tb_find_dp_out(struct tb *tb, struct tb_port *in) 
	return NULL;

}



static bool tb_tunnel_one_dp(struct tb *tb)

static bool tb_tunnel_one_dp(struct tb *tb, struct tb_port *in,

			     struct tb_port *out)

{

	int available_up, available_down, ret, link_nr;

	struct tb_cm *tcm = tb_priv(tb);

	struct tb_port *port, *in, *out;

	int consumed_up, consumed_down;

	struct tb_tunnel *tunnel;



	/*

	 * Find pair of inactive DP IN and DP OUT adapters and then

	 * establish a DP tunnel between them.

	 */

	tb_dbg(tb, "looking for DP IN <-> DP OUT pairs:\n");



	in = NULL;

	out = NULL;

	list_for_each_entry(port, &tcm->dp_resources, list) {

		if (!tb_port_is_dpin(port))

			continue;



		if (tb_port_is_enabled(port)) {

			tb_port_dbg(port, "DP IN in use\n");

			continue;

		}



		in = port;

		tb_port_dbg(in, "DP IN available\n");



		out = tb_find_dp_out(tb, port);

		if (out)

			break;

	}



	if (!in) {

		tb_dbg(tb, "no suitable DP IN adapter available, not tunneling\n");

		return false;

	}

	if (!out) {

		tb_dbg(tb, "no suitable DP OUT adapter available, not tunneling\n");

		return false;

	}



	/*

	 * This is only applicable to links that are not bonded (so

	 * when Thunderbolt 1 hardware is involved somewhere in the
@@ -1801,14 +1767,18 @@ static bool tb_tunnel_one_dp(struct tb *tb) 
		goto err_free;

	}



	/* If fail reading tunnel's consumed bandwidth, tear it down */

	ret = tb_tunnel_consumed_bandwidth(tunnel, &consumed_up, &consumed_down);

	if (ret)

		goto err_deactivate;



	list_add_tail(&tunnel->list, &tcm->tunnel_list);

	tb_reclaim_usb3_bandwidth(tb, in, out);



	tb_reclaim_usb3_bandwidth(tb, in, out);

	/*

	 * Transition the links to asymmetric if the consumption exceeds

	 * the threshold.

	 */

	if (!tb_tunnel_consumed_bandwidth(tunnel, &consumed_up, &consumed_down))

	tb_configure_asym(tb, in, out, consumed_up, consumed_down);



	/* Update the domain with the new bandwidth estimation */
@@ -1821,6 +1791,8 @@ static bool tb_tunnel_one_dp(struct tb *tb) 
	tb_increase_tmu_accuracy(tunnel);

	return true;



err_deactivate:

	tb_tunnel_deactivate(tunnel);

err_free:

	tb_tunnel_free(tunnel);

err_reclaim_usb:
@@ -1840,13 +1812,43 @@ static bool tb_tunnel_one_dp(struct tb *tb) 


static void tb_tunnel_dp(struct tb *tb)

{

	struct tb_cm *tcm = tb_priv(tb);

	struct tb_port *port, *in, *out;



	if (!tb_acpi_may_tunnel_dp()) {

		tb_dbg(tb, "DP tunneling disabled, not creating tunnel\n");

		return;

	}



	while (tb_tunnel_one_dp(tb))

		;

	/*

	 * Find pair of inactive DP IN and DP OUT adapters and then

	 * establish a DP tunnel between them.

	 */

	tb_dbg(tb, "looking for DP IN <-> DP OUT pairs:\n");



	in = NULL;

	out = NULL;

	list_for_each_entry(port, &tcm->dp_resources, list) {

		if (!tb_port_is_dpin(port))

			continue;



		if (tb_port_is_enabled(port)) {

			tb_port_dbg(port, "DP IN in use\n");

			continue;

		}



		in = port;

		tb_port_dbg(in, "DP IN available\n");



		out = tb_find_dp_out(tb, port);

		if (out)

			tb_tunnel_one_dp(tb, in, out);

		else

			tb_port_dbg(in, "no suitable DP OUT adapter available, not tunneling\n");

	}



	if (!in)

		tb_dbg(tb, "no suitable DP IN adapter available, not tunneling\n");

}



static void tb_enter_redrive(struct tb_port *port)