Received: by 2002:a25:8b91:0:0:0:0:0 with SMTP id j17csp1950639ybl; Sun, 19 Jan 2020 15:17:53 -0800 (PST) X-Google-Smtp-Source: APXvYqyxRFXKNtUYseMXEi+hYMawFWBp8pO+l9JPDlO4VKtes0yLhfYp36NwsIOo58unF3ubwixr X-Received: by 2002:a54:4e8d:: with SMTP id c13mr10920362oiy.27.1579475872634; Sun, 19 Jan 2020 15:17:52 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1579475872; cv=none; d=google.com; s=arc-20160816; b=ZB6ZdouPbah3dckLn7FtYWfmY5BtH5bNiFDXKVxmeO3IIwqlV7S1Lvb+TC6wKa5PFv U1ceCHfuAkXIh379zmV8x1KF/XFWi4mGiOmRPVUPRAlVRnhQcN1PCkdHhNwr/vcz6jdp lDQ5T98eAeQhobE9Qi3xKaNt+9URtxcBdCoJx84oQH5bwBPc1z++JAcwraUTbXsc/uYh g3VcXQwlbitgJQ49Dv0IZRlmLn4viMWlao3CLNmPUf18rLGcR52CcMUsSQAliXB2M1qr vl26LPfQeCIkG2+LEhuTVuQqSNCzW38jQotnNox7ml0E6Akmn7ldLy55aO2FouN1a04V kXXQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:date:subject:from:message-id; bh=E9U/spfbQRyPHBEgi1xBlyNQ5+vpfdnwjjR00PepuU4=; b=uC6QQcqihiFgCDBNzP4jDS2t9LcTYsHiAxqTI1FB+uP79sF/7I3+CrflgW+xLEPfxX w9OZ1YAGjDXyu95X6C5uivy9jfLbowd4VvVnIs/CCka8cFC2UXMHzZMmhm5hvd8B+zGd 6xEDwJZ5n5rT+AO9N1ReYsrt4OMHRj+Lc4pONk0H0DOdffKy/z6P4j7jQG9TQi4rbEML 3mpWZJAK1eLe/1Yrlv4T2QsrJBfxF/KpPw/Y2Rs3DIY+PovhIjlQPvgZO+rpP0md0RDH 53kJ79WXEjtkoPuCz1U7zr0ECCicop/Br3JNOb4mxbo4Cq2Ffwf/0wdROL4V3L5JhIBs sqPQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id b67si8141536oii.79.2020.01.19.15.17.40; Sun, 19 Jan 2020 15:17:52 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729106AbgASXQd (ORCPT + 99 others); Sun, 19 Jan 2020 18:16:33 -0500 Received: from kvm5.telegraphics.com.au ([98.124.60.144]:49722 "EHLO kvm5.telegraphics.com.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728841AbgASXQb (ORCPT ); Sun, 19 Jan 2020 18:16:31 -0500 Received: by kvm5.telegraphics.com.au (Postfix, from userid 502) id A042A28ED2; Sun, 19 Jan 2020 18:16:30 -0500 (EST) Message-Id: From: Finn Thain Subject: [PATCH net 00/19] Fixes for SONIC ethernet driver Date: Mon, 20 Jan 2020 09:56:09 +1100 To: "David S. Miller" Cc: Thomas Bogendoerfer , Chris Zankel , Laurent Vivier , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi David, Various SONIC driver problems have become apparent over the years, including tx watchdog timeouts, lost packets and duplicated packets. The problems are mostly caused by bugs in buffer handling, locking and (re-)initialization code. This patch series resolves these problems. Several cleanup patches are included at the beginning. This series has been tested on National Semiconductor hardware (macsonic), qemu-system-m68k (macsonic) and qemu-system-mips64el (jazzsonic). The emulated dp8393x device used in QEMU also has bugs. I have fixed the bugs that I know of in a series of patches at, https://github.com/fthain/qemu/commits/sonic Finn Thain (19): net/sonic: Remove obsolete comment net/sonic: Remove redundant next_tx variable net/sonic: Refactor duplicated code net/sonic: Add mutual exclusion for accessing shared state net/sonic: Remove redundant netif_start_queue() call net/macsonic: Remove interrupt handler wrapper net/sonic: Clear interrupt flags immediately net/sonic: Use MMIO accessors net/sonic: Remove explicit memory barriers net/sonic: Start packet transmission immediately net/sonic: Fix interface error stats collection net/sonic: Fix receive buffer handling net/sonic: Avoid needless receive descriptor EOL flag updates net/sonic: Improve receive descriptor status flag check net/sonic: Fix receive buffer replenishment net/sonic: Quiesce SONIC before re-initializing descriptor memory net/sonic: Fix command register usage net/sonic: Fix CAM initialization net/sonic: Prevent tx watchdog timeout drivers/net/ethernet/natsemi/jazzsonic.c | 31 +- drivers/net/ethernet/natsemi/macsonic.c | 48 +-- drivers/net/ethernet/natsemi/sonic.c | 433 ++++++++++++++--------- drivers/net/ethernet/natsemi/sonic.h | 45 ++- drivers/net/ethernet/natsemi/xtsonic.c | 40 +-- 5 files changed, 313 insertions(+), 284 deletions(-) -- 2.24.1