Re: [PATCH net] ibmvnic: Warn unknown speed message only when carrier is present

[Thomas Falcon <tlfalcon@linux.ibm.com>]

On 9/16/19 9:50 AM, Murilo Fossa Vicentini wrote:
> With commit 0655f9943df2 ("net/ibmvnic: Update carrier state after link
> state change") we are now able to detect when the carrier is properly
> present in the device, so only report an unexpected unknown speed when it
> is properly detected. Unknown speed is expected to be seen by the device
> in case the backing device has no link detected.
>
> Reported-by: Abdul Haleem <abdhalee@linux.vnet.ibm.com>
> Tested-by: Abdul Haleem <abdhalee@linux.vnet.ibm.com>
> Signed-off-by: Murilo Fossa Vicentini <muvic@linux.ibm.com>
> ---

Thanks, Murilo!

Reviewed-by: Thomas Falcon <tlfalcon@linux.ibm.com>

>   drivers/net/ethernet/ibm/ibmvnic.c | 7 ++++---
>   1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/ethernet/ibm/ibmvnic.c b/drivers/net/ethernet/ibm/ibmvnic.c
> index 5cb55ea671e3..3a6725daf7dc 100644
> --- a/drivers/net/ethernet/ibm/ibmvnic.c
> +++ b/drivers/net/ethernet/ibm/ibmvnic.c
> @@ -4312,13 +4312,14 @@ static int handle_query_phys_parms_rsp(union ibmvnic_crq *crq,
>   {
>   	struct net_device *netdev = adapter->netdev;
>   	int rc;
> +	__be32 rspeed = cpu_to_be32(crq->query_phys_parms_rsp.speed);
>   
>   	rc = crq->query_phys_parms_rsp.rc.code;
>   	if (rc) {
>   		netdev_err(netdev, "Error %d in QUERY_PHYS_PARMS\n", rc);
>   		return rc;
>   	}
> -	switch (cpu_to_be32(crq->query_phys_parms_rsp.speed)) {
> +	switch (rspeed) {
>   	case IBMVNIC_10MBPS:
>   		adapter->speed = SPEED_10;
>   		break;
> @@ -4344,8 +4345,8 @@ static int handle_query_phys_parms_rsp(union ibmvnic_crq *crq,
>   		adapter->speed = SPEED_100000;
>   		break;
>   	default:
> -		netdev_warn(netdev, "Unknown speed 0x%08x\n",
> -			    cpu_to_be32(crq->query_phys_parms_rsp.speed));
> +		if (netif_carrier_ok(netdev))
> +			netdev_warn(netdev, "Unknown speed 0x%08x\n", rspeed);
>   		adapter->speed = SPEED_UNKNOWN;
>   	}
>   	if (crq->query_phys_parms_rsp.flags1 & IBMVNIC_FULL_DUPLEX)