Commit 7f936f2a authored by David S. Miller's avatar David S. Miller
Browse files

Merge branch 's390-qeth-fixes' 



Julian Wiedmann says:

====================
s390/qeth: fixes 2019-12-23

please apply the following patch series for qeth to your net tree.

This brings two fixes for errors during device initialization, deals with
several issues in the vnicc control code, and adds a missing lock.
====================

Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents 47d0b2fe 0b698c83

drivers/s390/net/qeth_core_main.c

+9 −20
Original line number Diff line number Diff line
@@ -2482,50 +2482,46 @@ static int qeth_mpc_initialize(struct qeth_card *card) 
	rc = qeth_cm_enable(card);

	if (rc) {

		QETH_CARD_TEXT_(card, 2, "2err%d", rc);

		goto out_qdio;

		return rc;

	}

	rc = qeth_cm_setup(card);

	if (rc) {

		QETH_CARD_TEXT_(card, 2, "3err%d", rc);

		goto out_qdio;

		return rc;

	}

	rc = qeth_ulp_enable(card);

	if (rc) {

		QETH_CARD_TEXT_(card, 2, "4err%d", rc);

		goto out_qdio;

		return rc;

	}

	rc = qeth_ulp_setup(card);

	if (rc) {

		QETH_CARD_TEXT_(card, 2, "5err%d", rc);

		goto out_qdio;

		return rc;

	}

	rc = qeth_alloc_qdio_queues(card);

	if (rc) {

		QETH_CARD_TEXT_(card, 2, "5err%d", rc);

		goto out_qdio;

		return rc;

	}

	rc = qeth_qdio_establish(card);

	if (rc) {

		QETH_CARD_TEXT_(card, 2, "6err%d", rc);

		qeth_free_qdio_queues(card);

		goto out_qdio;

		return rc;

	}

	rc = qeth_qdio_activate(card);

	if (rc) {

		QETH_CARD_TEXT_(card, 2, "7err%d", rc);

		goto out_qdio;

		return rc;

	}

	rc = qeth_dm_act(card);

	if (rc) {

		QETH_CARD_TEXT_(card, 2, "8err%d", rc);

		goto out_qdio;

		return rc;

	}



	return 0;

out_qdio:

	qeth_qdio_clear_card(card, !IS_IQD(card));

	qdio_free(CARD_DDEV(card));

	return rc;

}



void qeth_print_status_message(struct qeth_card *card)
@@ -3429,11 +3425,6 @@ int qeth_configure_cq(struct qeth_card *card, enum qeth_cq cq) 
			goto out;

		}



		if (card->state != CARD_STATE_DOWN) {

			rc = -1;

			goto out;

		}



		qeth_free_qdio_queues(card);

		card->options.cq = cq;

		rc = 0;
@@ -5035,10 +5026,8 @@ retriable: 
	}

	if (qeth_adp_supported(card, IPA_SETADP_SET_DIAG_ASSIST)) {

		rc = qeth_query_setdiagass(card);

		if (rc < 0) {

		if (rc)

			QETH_CARD_TEXT_(card, 2, "8err%d", rc);

			goto out;

		}

	}



	if (!qeth_is_diagass_supported(card, QETH_DIAGS_CMD_TRAP) ||

drivers/s390/net/qeth_l2_main.c

+5 −5
Original line number Diff line number Diff line
@@ -287,12 +287,12 @@ static void qeth_l2_stop_card(struct qeth_card *card) 
		card->state = CARD_STATE_HARDSETUP;

	}

	if (card->state == CARD_STATE_HARDSETUP) {

		qeth_qdio_clear_card(card, 0);

		qeth_drain_output_queues(card);

		qeth_clear_working_pool_list(card);

		card->state = CARD_STATE_DOWN;

	}



	qeth_qdio_clear_card(card, 0);

	flush_workqueue(card->event_wq);

	card->info.mac_bits &= ~QETH_LAYER2_MAC_REGISTERED;

	card->info.promisc_mode = 0;
@@ -1952,8 +1952,7 @@ int qeth_l2_vnicc_get_timeout(struct qeth_card *card, u32 *timeout) 
/* check if VNICC is currently enabled */

bool qeth_l2_vnicc_is_in_use(struct qeth_card *card)

{

	/* if everything is turned off, VNICC is not active */

	if (!card->options.vnicc.cur_chars)

	if (!card->options.vnicc.sup_chars)

		return false;

	/* default values are only OK if rx_bcast was not enabled by user

	 * or the card is offline.
@@ -2040,8 +2039,9 @@ static void qeth_l2_vnicc_init(struct qeth_card *card) 
	/* enforce assumed default values and recover settings, if changed  */

	error |= qeth_l2_vnicc_recover_timeout(card, QETH_VNICC_LEARNING,

					       timeout);

	chars_tmp = card->options.vnicc.wanted_chars ^ QETH_VNICC_DEFAULT;

	chars_tmp |= QETH_VNICC_BRIDGE_INVISIBLE;

	/* Change chars, if necessary  */

	chars_tmp = card->options.vnicc.wanted_chars ^

		    card->options.vnicc.cur_chars;

	chars_len = sizeof(card->options.vnicc.wanted_chars) * BITS_PER_BYTE;

	for_each_set_bit(i, &chars_tmp, chars_len) {

		vnicc = BIT(i);

drivers/s390/net/qeth_l3_main.c

+1 −1
Original line number Diff line number Diff line
@@ -1307,12 +1307,12 @@ static void qeth_l3_stop_card(struct qeth_card *card) 
		card->state = CARD_STATE_HARDSETUP;

	}

	if (card->state == CARD_STATE_HARDSETUP) {

		qeth_qdio_clear_card(card, 0);

		qeth_drain_output_queues(card);

		qeth_clear_working_pool_list(card);

		card->state = CARD_STATE_DOWN;

	}



	qeth_qdio_clear_card(card, 0);

	flush_workqueue(card->event_wq);

	card->info.promisc_mode = 0;

}

drivers/s390/net/qeth_l3_sys.c

+28 −12
Original line number Diff line number Diff line
@@ -242,21 +242,33 @@ static ssize_t qeth_l3_dev_hsuid_store(struct device *dev, 
		struct device_attribute *attr, const char *buf, size_t count)

{

	struct qeth_card *card = dev_get_drvdata(dev);

	int rc = 0;

	char *tmp;

	int rc;



	if (!IS_IQD(card))

		return -EPERM;

	if (card->state != CARD_STATE_DOWN)

		return -EPERM;

	if (card->options.sniffer)

		return -EPERM;

	if (card->options.cq == QETH_CQ_NOTAVAILABLE)

		return -EPERM;



	mutex_lock(&card->conf_mutex);

	if (card->state != CARD_STATE_DOWN) {

		rc = -EPERM;

		goto out;

	}



	if (card->options.sniffer) {

		rc = -EPERM;

		goto out;

	}



	if (card->options.cq == QETH_CQ_NOTAVAILABLE) {

		rc = -EPERM;

		goto out;

	}



	tmp = strsep((char **)&buf, "\n");

	if (strlen(tmp) > 8)

		return -EINVAL;

	if (strlen(tmp) > 8) {

		rc = -EINVAL;

		goto out;

	}



	if (card->options.hsuid[0])

		/* delete old ip address */
@@ -267,11 +279,13 @@ static ssize_t qeth_l3_dev_hsuid_store(struct device *dev, 
		card->options.hsuid[0] = '\0';

		memcpy(card->dev->perm_addr, card->options.hsuid, 9);

		qeth_configure_cq(card, QETH_CQ_DISABLED);

		return count;

		goto out;

	}



	if (qeth_configure_cq(card, QETH_CQ_ENABLED))

		return -EPERM;

	if (qeth_configure_cq(card, QETH_CQ_ENABLED)) {

		rc = -EPERM;

		goto out;

	}



	snprintf(card->options.hsuid, sizeof(card->options.hsuid),

		 "%-8s", tmp);
@@ -280,6 +294,8 @@ static ssize_t qeth_l3_dev_hsuid_store(struct device *dev, 


	rc = qeth_l3_modify_hsuid(card, true);



out:

	mutex_unlock(&card->conf_mutex);

	return rc ? rc : count;

}

CRA Git | Maintained and supported by SUSTech CRA and CCSE