Re: [PATCH] netfilter: conntrack: fix -Wformat

2020-12-13 Thread Tom Rix
On 12/2/20 2:34 PM, Nick Desaulniers wrote: > On Tue, Nov 10, 2020 at 2:04 PM Joe Perches wrote: >> On Tue, 2020-11-10 at 14:00 -0800, Nick Desaulniers wrote: >> >>> Yeah, we could go through and remove %h and %hh to solve this, too, right? >> Yup. >> >> I think one of the checkpatch improvement

Re: [PATCH] netfilter: conntrack: fix -Wformat

2020-12-13 Thread Tom Rix
On 12/13/20 3:25 PM, Joe Perches wrote: > On Sun, 2020-12-13 at 11:21 -0800, Tom Rix wrote: >> On 12/2/20 2:34 PM, Nick Desaulniers wrote: >>> On Tue, Nov 10, 2020 at 2:04 PM Joe Perches wrote: >>>> On Tue, 2020-11-10 at 14:00 -0800, Nick Desaulniers wrote: >&g

Re: [PATCH] atm: ambassador: remove h from printk format specifier

2020-12-17 Thread Tom Rix
On 12/16/20 4:45 PM, Jakub Kicinski wrote: > On Tue, 15 Dec 2020 06:22:28 -0800 t...@redhat.com wrote: >> From: Tom Rix >> >> See Documentation/core-api/printk-formats.rst. >> h should no longer be used in the format specifier for printk. >> >> Signed-o

Re: [PATCH] atm: ambassador: remove h from printk format specifier

2020-12-17 Thread Tom Rix
On 12/17/20 9:28 AM, Jakub Kicinski wrote: > On Thu, 17 Dec 2020 05:17:24 -0800 Tom Rix wrote: >> On 12/16/20 4:45 PM, Jakub Kicinski wrote: >>> On Tue, 15 Dec 2020 06:22:28 -0800 t...@redhat.com wrote: >>>> From: Tom Rix >>>> >>>> See Do

Re: [PATCH] net: bna: remove trailing semicolon in macro definition

2020-12-01 Thread Tom Rix
On 11/30/20 7:01 PM, Jakub Kicinski wrote: > On Fri, 27 Nov 2020 08:55:50 -0800 t...@redhat.com wrote: >> From: Tom Rix >> >> The macro use will already have a semicolon. >> >> Signed-off-by: Tom Rix >> --- >> drivers/net/ethernet/brocade/bna

Re: [PATCH] netfilter: conntrack: fix -Wformat

2020-12-02 Thread Tom Rix
On 12/2/20 2:34 PM, Nick Desaulniers wrote: > On Tue, Nov 10, 2020 at 2:04 PM Joe Perches wrote: >> On Tue, 2020-11-10 at 14:00 -0800, Nick Desaulniers wrote: >> >>> Yeah, we could go through and remove %h and %hh to solve this, too, right? >> Yup. >> >> I think one of the checkpatch improvement

Re: [RFC] MAINTAINERS tag for cleanup robot

2020-11-22 Thread Tom Rix
On 11/21/20 7:23 PM, Matthew Wilcox wrote: > On Sat, Nov 21, 2020 at 08:50:58AM -0800, t...@redhat.com wrote: >> The fixer review is >> https://reviews.llvm.org/D91789 >> >> A run over allyesconfig for x86_64 finds 62 issues, 5 are false positives. >> The false positives are caused by macros pass

Re: [RFC] MAINTAINERS tag for cleanup robot

2020-11-22 Thread Tom Rix
On 11/22/20 6:56 AM, Matthew Wilcox wrote: > On Sun, Nov 22, 2020 at 06:46:46AM -0800, Tom Rix wrote: >> On 11/21/20 7:23 PM, Matthew Wilcox wrote: >>> On Sat, Nov 21, 2020 at 08:50:58AM -0800, t...@redhat.com wrote: >>>> The fixer review is >>>> https

Re: [RFC] MAINTAINERS tag for cleanup robot

2020-11-22 Thread Tom Rix
On 11/21/20 9:10 AM, Joe Perches wrote: > On Sat, 2020-11-21 at 08:50 -0800, t...@redhat.com wrote: >> A difficult part of automating commits is composing the subsystem >> preamble in the commit log. For the ongoing effort of a fixer producing >> one or two fixes a release the use of 'treewide:'

Re: [RFC] MAINTAINERS tag for cleanup robot

2020-11-23 Thread Tom Rix
On 11/22/20 10:22 AM, Joe Perches wrote: > On Sun, 2020-11-22 at 08:33 -0800, Tom Rix wrote: >> On 11/21/20 9:10 AM, Joe Perches wrote: >>> On Sat, 2020-11-21 at 08:50 -0800, t...@redhat.com wrote: >>>> A difficult part of automating commits is composing the subsyst

Re: [PATCH v5 0/9] "Task_isolation" mode

2020-11-24 Thread Tom Rix
On 11/23/20 9:42 AM, Alex Belits wrote: > This is an update of task isolation work that was originally done by > Chris Metcalf and maintained by him until > November 2017. It is adapted to the current kernel and cleaned up to > implement its functionality in a more complete and cleaner manner.

Re: [PATCH] NFS: remove trailing semicolon in macro definition

2020-11-29 Thread Tom Rix
On 11/29/20 8:50 AM, Trond Myklebust wrote: > On Sun, 2020-11-29 at 16:42 +, Trond Myklebust wrote: >> Hi Tom, >> >> On Fri, 2020-11-27 at 11:43 -0800, t...@redhat.com wrote: >>> From: Tom Rix >>> >>> The macro use will already

Re: [PATCH] bpf: remove trailing semicolon in macro definition

2020-11-29 Thread Tom Rix
On 11/27/20 4:54 PM, Daniel Borkmann wrote: > On 11/27/20 8:27 PM, t...@redhat.com wrote: >> From: Tom Rix >> >> The macro use will already have a semicolon. >> >> Signed-off-by: Tom Rix >> --- >>   include/trace/events/xdp.h | 4 ++-- >

Re: Subject: [RFC] clang tooling cleanups

2020-11-10 Thread Tom Rix
On 11/9/20 6:52 PM, Joe Perches wrote: > On Tue, 2020-10-27 at 09:42 -0700, t...@redhat.com wrote: >> This rfc will describe >> An upcoming treewide cleanup. >> How clang tooling was used to programatically do the clean up. >> Solicit opinions on how to generally use clang tooling. >> >> The clan

Re: [PATCH] amd-xgbe: remove h from printk format specifier

2020-12-23 Thread Tom Rix
On 12/23/20 12:14 PM, Joe Perches wrote: > On Wed, 2020-12-23 at 11:43 -0800, t...@redhat.com wrote: >> From: Tom Rix >> >> This change fixes the checkpatch warning described in this commit >> commit cbacb5ab0aa0 ("docs: printk-formats: Stop encouraging use of

Re: [PATCH] nfp: remove h from printk format specifier

2020-12-24 Thread Tom Rix
On 12/24/20 12:21 PM, Simon Horman wrote: > On Wed, Dec 23, 2020 at 12:20:53PM -0800, t...@redhat.com wrote: >> From: Tom Rix >> >> This change fixes the checkpatch warning described in this commit >> commit cbacb5ab0aa0 ("docs: printk-formats: Stop encouragin

Re: [PATCH] nfp: remove h from printk format specifier

2020-12-25 Thread Tom Rix
On 12/24/20 2:39 PM, Joe Perches wrote: > On Thu, 2020-12-24 at 14:14 -0800, Tom Rix wrote: >> On 12/24/20 12:21 PM, Simon Horman wrote: >>> On Wed, Dec 23, 2020 at 12:20:53PM -0800, t...@redhat.com wrote: >>>> From: Tom Rix >>>> >>>> This

Re: [PATCH] nfp: remove h from printk format specifier

2020-12-25 Thread Tom Rix
On 12/25/20 9:06 AM, Joe Perches wrote: > On Fri, 2020-12-25 at 06:56 -0800, Tom Rix wrote: >> On 12/24/20 2:39 PM, Joe Perches wrote: > [] >>> Kernel code doesn't use a signed char or short with %hx or %hu very often >>> but in case you didn't already

Re: [PATCH] rxrpc: fix handling of an unsupported token type in rxrpc_read()

2021-01-04 Thread Tom Rix
On 1/4/21 4:50 AM, David Howells wrote: > t...@redhat.com wrote: > >> -switch (token->security_index) { >> -case RXRPC_SECURITY_RXKAD: >> ... >> -switch (token->security_index) { >> -case RXRPC_SECURITY_RXKAD: > These switches need to be kept. Ther

Re: [PATCH] rxrpc: fix handling of an unsupported token type in rxrpc_read()

2021-01-06 Thread Tom Rix
ing both loops log a message and return an error in this > case. This should only happen if a new token type is incompletely > implemented, so it should normally be impossible to trigger this. > > Fixes: 9a059cd5ca7d ("rxrpc: Downgrade the BUG() for unsupported t

Re: [PATCH] rxrpc: fix handling of an unsupported token type in rxrpc_read()

2021-01-06 Thread Tom Rix
On 1/6/21 11:44 AM, David Howells wrote: > Tom Rix wrote: > >> These two loops iterate over the same data, i believe returning here is all >> that is needed. > But if the first loop is made to support a new type, but the second loop is > missed, it will then likely oops

Re: [PATCH] rxrpc: fix handling of an unsupported token type in rxrpc_read()

2021-01-06 Thread Tom Rix
On 1/6/21 1:09 PM, David Howells wrote: > Tom Rix wrote: > >> On 1/6/21 11:44 AM, David Howells wrote: >>> Tom Rix wrote: >>> >>>> These two loops iterate over the same data, i believe returning here is all >>>> that is needed. >>&g

Re: [RFC] treewide: cleanup unreachable breaks

2020-10-18 Thread Tom Rix
On 10/17/20 10:43 PM, Greg KH wrote: > On Sat, Oct 17, 2020 at 09:09:28AM -0700, t...@redhat.com wrote: >> From: Tom Rix >> >> This is a upcoming change to clean up a new warning treewide. >> I am wondering if the change could be one mega patch (see below) or >>

Re: [PATCH] wireless: remove unneeded break

2020-10-19 Thread Tom Rix
On 10/19/20 9:20 AM, Joe Perches wrote: > On Mon, 2020-10-19 at 10:54 -0500, Gustavo A. R. Silva wrote: >> On 10/19/20 10:21, Joe Perches wrote: >>> On Mon, 2020-10-19 at 17:14 +0200, Christian Lamparter wrote: >>>> On 19/10/2020 17:05, t...@redhat.com wrote: >

Re: [RFC] treewide: cleanup unreachable breaks

2020-10-20 Thread Tom Rix
On 10/19/20 12:42 PM, Nick Desaulniers wrote: > On Sat, Oct 17, 2020 at 10:43 PM Greg KH wrote: >> On Sat, Oct 17, 2020 at 09:09:28AM -0700, t...@redhat.com wrote: >>> From: Tom Rix >>> >>> This is a upcoming change to clean up a new warning treewide. >&

Re: [RFC] treewide: cleanup unreachable breaks

2020-10-20 Thread Tom Rix
On 10/19/20 4:05 PM, Jason Gunthorpe wrote: > On Mon, Oct 19, 2020 at 12:42:15PM -0700, Nick Desaulniers wrote: >> On Sat, Oct 17, 2020 at 10:43 PM Greg KH wrote: >>> On Sat, Oct 17, 2020 at 09:09:28AM -0700, t...@redhat.com wrote: >>>> From: Tom Rix >>

Re: [RFC PATCH 2/6] fpga: dfl: export network configuration info for DFL based FPGA

2020-10-24 Thread Tom Rix
On 10/23/20 1:45 AM, Xu Yilun wrote: > This patch makes preparation for supporting DFL Ether Group private > feature driver, which reads bitstream_id.vendor_net_cfg field to > determin the interconnection of network components on FPGA device. > > Signed-off-by: Xu Yilun > --- > drivers/fpga/dfl

Re: [RFC PATCH 1/6] docs: networking: add the document for DFL Ether Group driver

2020-10-24 Thread Tom Rix
On 10/23/20 1:45 AM, Xu Yilun wrote: > This patch adds the document for DFL Ether Group driver. > > Signed-off-by: Xu Yilun > --- > .../networking/device_drivers/ethernet/index.rst | 1 + > .../ethernet/intel/dfl-eth-group.rst | 102 > + > 2 files changed,

Re: [RFC PATCH 3/6] fpga: dfl: add an API to get the base device for dfl device

2020-10-24 Thread Tom Rix
On 10/23/20 1:45 AM, Xu Yilun wrote: > This patch adds an API for dfl devices to find which physical device > owns the DFL. > > This patch makes preparation for supporting DFL Ether Group private > feature driver. It uses this information to determine which retimer > device physically connects to

Re: [RFC PATCH 4/6] ethernet: m10-retimer: add support for retimers on Intel MAX 10 BMC

2020-10-24 Thread Tom Rix
On 10/23/20 1:45 AM, Xu Yilun wrote: > This driver supports the ethernet retimers (Parkvale) for the Intel PAC > (Programmable Acceleration Card) N3000, which is a FPGA based Smart NIC. Parkvale is a code name, it would be better if the public name was used. As this is a physical chip that coul

Re: [RFC PATCH 5/6] ethernet: dfl-eth-group: add DFL eth group private feature driver

2020-10-24 Thread Tom Rix
On 10/23/20 1:45 AM, Xu Yilun wrote: > This driver supports the DFL Ether Group private feature for the Intel(R) > PAC N3000 FPGA Smart NIC. > > The DFL Ether Group private feature contains an Ether Wrapper and several > ports (phy-mac pair). There are two types of Ether Groups, line side & (phy

Re: [RFC PATCH 4/6] ethernet: m10-retimer: add support for retimers on Intel MAX 10 BMC

2020-10-24 Thread Tom Rix
On 10/24/20 9:39 AM, Andrew Lunn wrote: > On Sat, Oct 24, 2020 at 08:03:51AM -0700, Tom Rix wrote: >> On 10/23/20 1:45 AM, Xu Yilun wrote: >>> This driver supports the ethernet retimers (Parkvale) for the Intel PAC >>> (Programmable Acceleration Card) N3000, whic

Re: [RFC PATCH 6/6] ethernet: dfl-eth-group: add support for the 10G configurations

2020-10-24 Thread Tom Rix
On 10/23/20 1:45 AM, Xu Yilun wrote: > This patch adds 10G configurations support for dfl ether group private > feature. > > 10G configurations have different PHY & MAC IP blocks from 25G, so a > different set of HW operations is implemented, but the software arch is > quite similar with 25G. Ye

Re: [PATCH] net: cls_api: remove unneeded local variable in tc_dump_chain()

2020-10-28 Thread Tom Rix
On 10/28/20 4:35 AM, Lukas Bulwahn wrote: > make clang-analyzer on x86_64 defconfig caught my attention with: > > net/sched/cls_api.c:2964:3: warning: Value stored to 'parent' is never read > [clang-analyzer-deadcode.DeadStores] > parent = 0; > ^ > > net/sched/cl

Re: [RFC v2 3/5] arm64: socfpga: rename ARCH_STRATIX10 to ARCH_SOCFPGA64

2021-03-10 Thread Tom Rix
On 3/10/21 1:45 AM, Lee Jones wrote: > On Wed, 10 Mar 2021, Krzysztof Kozlowski wrote: > >> Prepare for merging Stratix 10, Agilex and N5X into one arm64 >> architecture by first renaming the ARCH_STRATIX10 into ARCH_SOCFPGA64. >> >> The existing ARCH_SOCFPGA (in ARMv7) Kconfig symbol cannot be u

Re: [PATCH v3 00/15] arm64 / clk: socfpga: simplifying, cleanups and compile testing

2021-03-11 Thread Tom Rix
On 3/11/21 7:25 AM, Krzysztof Kozlowski wrote: > Hi, > > All three Intel arm64 SoCFPGA architectures (Agilex, N5X and Stratix 10) > are basically flavors/platforms of the same architecture. At least from > the Linux point of view. Up to a point that N5X and Agilex share DTSI. > Having three top

Re: [PATCH net-next] airo: work around stack usage warning

2021-03-23 Thread Tom Rix
On 3/23/21 6:16 AM, Arnd Bergmann wrote: > From: Arnd Bergmann > > gcc-11 with KASAN on 32-bit arm produces a warning about a function > that needs a lot of stack space: > > drivers/net/wireless/cisco/airo.c: In function 'setup_card.constprop': > drivers/net/wireless/cisco/airo.c:3960:1: error:

Re: [PATCH] ptp: ptp_clockmatrix: initialize variables

2020-10-12 Thread Tom Rix
On 10/12/20 3:01 PM, Richard Cochran wrote: > On Sun, Oct 11, 2020 at 01:09:55PM -0700, t...@redhat.com wrote: >> From: Tom Rix >> >> Clang static analysis reports this representative problem >> >> ptp_clockmatrix.c:1852:2: warning: 5th function call argume

Re: [RFC PATCH 00/17] Drop uses of pci_read_config_*() return value

2020-08-02 Thread Tom Rix
On 8/1/20 5:56 AM, Borislav Petkov wrote: > On Sat, Aug 01, 2020 at 01:24:29PM +0200, Saheed O. Bolarinwa wrote: >> The return value of pci_read_config_*() may not indicate a device error. >> However, the value read by these functions is more likely to indicate >> this kind of error. This present

Re: [PATCH] bpf: BPF_SYSCALL depends INET

2020-07-23 Thread Tom Rix
On 7/23/20 8:27 AM, Alexei Starovoitov wrote: > On Thu, Jul 23, 2020 at 7:19 AM wrote: >> From: Tom Rix >> >> A link error >> >> kernel/bpf/net_namespace.o: In function `bpf_netns_link_release': >> net_namespace.c: undefined reference to `bpf_sk_lo

Re: [PATCH] net: openvswitch: pass NULL for unused parameters

2020-08-30 Thread Tom Rix
On 8/30/20 1:02 PM, Andy Shevchenko wrote: > On Sun, Aug 30, 2020 at 6:17 PM wrote: >> From: Tom Rix >> >> clang static analysis flags these problems >> >> flow_table.c:713:2: warning: The expression is an uninitialized >> value.

Re: [PATCH v2] net: openvswitch: pass NULL for unused parameters

2020-09-02 Thread Tom Rix
On 9/1/20 1:11 PM, David Miller wrote: > From: t...@redhat.com > Date: Sun, 30 Aug 2020 14:26:30 -0700 > >> Passing unused parameters is a waste. > Poorly predicted branches are an even bigger waste. > > I'm not a big fan of this change and others have asked for performance > analysis to be perfo

Re: PATCH SB1250 NAPI support

2006-07-01 Thread Tom Rix
AIL PROTECTED]> wrote: Tom Rix <[EMAIL PROTECTED]> : [...] diff -rup a/drivers/net/sb1250-mac.c b/drivers/net/sb1250-mac.c --- a/drivers/net/sb1250-mac.c 2006-03-09 04:25:41.0 -0600 +++ b/drivers/net/sb1250-mac.c 2006-03-09 05:30:52.0 -0600 [...] @@ -2079,13 +2095,

PATCH SB1250 NAPI support

2006-06-04 Thread Tom Rix
. Tom On Thu, 25 May 2006 08:35:05 -0500, Martin Michlmayr <[EMAIL PROTECTED]> wrote: * Tom Rix <[EMAIL PROTECTED]> [2006-05-25 08:06]: I am busy now but will get to it sometime. It will require retesting with the latest kernel source. Is there a particular maintainer(s) that I

[PATCH] Broadcom Sibyte SB1xxx NAPI ethernet support

2006-03-19 Thread Tom Rix
This patch adds NAPI support for the Broadcom Sibyte SB1xxx family. The changes are limited to adding a new config key SBMAC_NAPI to the drivers/net/Kconfig and by adding the poll op and interrupt support to drivers/net/sb1250-mac.c. This patch also has a fix to drivers/net/sb1250-mac.c, the dma

Re: [PATCH] Broadcom Sibyte SB1xxx NAPI ethernet support

2006-03-21 Thread Tom Rix
Yes. They are soon to follow. Tom On Sun, 19 Mar 2006 17:08:23 -0600, Lennert Buytenhek <[EMAIL PROTECTED]> wrote: On Sun, Mar 19, 2006 at 05:12:32PM -0600, Tom Rix wrote: This patch also has a fix to drivers/net/sb1250-mac.c, the dma descriptor table ptr is allocated, aligned a

Re: [PATCH] Broadcom Sibyte SB1xxx NAPI ethernet support

2006-03-21 Thread Tom Rix
. Mark Mason from Broadcom was very helpful and tested this patch on at least a 1480. Tom On Sun, 19 Mar 2006 17:08:23 -0600, Lennert Buytenhek <[EMAIL PROTECTED]> wrote: On Sun, Mar 19, 2006 at 05:12:32PM -0600, Tom Rix wrote: This patch also has a fix to drivers/net/sb1250-mac.c, t

[PATCH] Broadcom Sibyte SB1xxx save unaligned dma descriptor pointer fix

2006-03-21 Thread Tom Rix
so that it could be properly freed. Tom On Sun, 19 Mar 2006 17:08:23 -0600, Lennert Buytenhek <[EMAIL PROTECTED]> wrote: On Sun, Mar 19, 2006 at 05:12:32PM -0600, Tom Rix wrote: This patch also has a fix to drivers/net/sb1250-mac.c, the dma descriptor table ptr is allocated, align