From owner-svn-src-head@freebsd.org Sat May 25 00:38:55 2019 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id C6EFB15B8F95; Sat, 25 May 2019 00:38:55 +0000 (UTC) (envelope-from cy.schubert@cschubert.com) Received: from smtp-out-no.shaw.ca (smtp-out-no.shaw.ca [64.59.134.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "Client", Issuer "CA" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 2854F8673C; Sat, 25 May 2019 00:38:54 +0000 (UTC) (envelope-from cy.schubert@cschubert.com) Received: from spqr.komquats.com ([70.67.125.17]) by shaw.ca with ESMTPA id UKi1h4qaisAGkUKi3hDEoN; Fri, 24 May 2019 18:38:52 -0600 X-Authority-Analysis: v=2.3 cv=WeVylHpX c=1 sm=1 tr=0 a=VFtTW3WuZNDh6VkGe7fA3g==:117 a=VFtTW3WuZNDh6VkGe7fA3g==:17 a=jpOVt7BSZ2e4Z31A5e1TngXxSK0=:19 a=E5NmQfObTbMA:10 a=ASrO2z8EAAAA:8 a=6I5d2MoRAAAA:8 a=YxBL1-UpAAAA:8 a=UEpEsKYdWQD3eQkOumkA:9 a=Rc5TUw_wQp4Z_7gz:21 a=eqxhlmoB-vvMXGhw:21 a=QEXdDO2ut3YA:10 a=i8UNxtqY09YYJATF_ugA:9 a=HIt_kXhqRHWN5h8v:21 a=bn3e2i_yJ9W2f3Ef:21 a=ci6mo512LUvLpNOh:21 a=_W_S_7VecoQA:10 a=hMlqHZxbFqNCaWTKXPQf:22 a=IjZwj45LgO3ly-622nXo:22 a=Ia-lj3WSrqcvXOmTRaiG:22 Received: from [10.224.108.1] (unknown [24.244.23.196]) by spqr.komquats.com (Postfix) with ESMTPSA id B5580CEB; Fri, 24 May 2019 17:38:48 -0700 (PDT) Date: Fri, 24 May 2019 17:38:24 -0700 User-Agent: K-9 Mail for Android In-Reply-To: References: <201905242233.x4OMXEAr071844@repo.freebsd.org> MIME-Version: 1.0 Subject: Re: svn commit: r348255 - head/sys/kern To: Steven Hartland , Conrad Meyer CC: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org From: Cy Schubert Message-ID: <003FD1FE-C765-4873-A784-D743F6A2156C@cschubert.com> X-CMAE-Envelope: MS4wfFoiywZUpU3etClt8mdHA81u4zTKgAC7wCs3mK9tf20KdMzco07r3xmQL9SdjK18gTjvuEOucZ8f9rwOgk61lAmJNa+w/+l7DIZTKFqMfDIzw6k4kZ14 hebWAjVWuJuWDTrvBUqcM2YW2Wwdt8sv1QtauY3ULpKRkyUmgMSWBbC2tLQQ59zDtZuBulqd0j5B2NNAdoyg8bcQN+GhBH9DVDAjRVrfLsRw2dN1leaDJh+8 GZZ30lY8yWegBmqeTkeeOMYrNOOK/ADlGnJrRs/2CEpORAHuq7X09LbaW6pwKUnUV22X/saf6Pj+HGYMVqdYod5Gog8cB0cq+ZjKIZfbbqI= X-Rspamd-Queue-Id: 2854F8673C X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-6.98 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-1.000,0]; NEURAL_HAM_SHORT(-0.98)[-0.977,0]; REPLY(-4.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000,0] Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.29 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 25 May 2019 00:38:56 -0000 +1 -------- Original Message -------- From: Steven Hartland Sent: May 24, 2019 4:21:39 PM PDT To: Conrad Meyer Cc: src-committers@freebsd=2Eorg, svn-src-all@freebsd=2Eorg, svn-src-head@= freebsd=2Eorg Subject: Re: svn commit: r348255 - head/sys/kern Just wanted to say I really appreciate the details in this commit message= =2E Its often the case the message get overlooked when it comes to the time needed to write a truly useful message to others and this a great example of the quality we should all try to follow=2E Regards Steve On Fri, 24 May 2019 at 23:33, Conrad Meyer wrote: > Author: cem > Date: Fri May 24 22:33:14 2019 > New Revision: 348255 > URL: https://svnweb=2Efreebsd=2Eorg/changeset/base/348255 > > Log: > Disable intr_storm_threshold mechanism by default > > The ixl=2E4 manual page has documented that the threshold falsely dete= cts > interrupt storms on 40Gbit NICs as long ago as 2015, and we have seen > similar false positives with the ioat(4) DMA device (which can push > GB/s)=2E > > For example, synthetic load can be generated with tools/tools/ioat > 'ioatcontrol 0 200 8192 1 1000' (allocate 200x8kB buffers, generate an > interrupt for each one, and do this for 1000 milliseconds)=2E With > storm-detection disabled, the Broadwell-EP version of this device is > capable > of generating ~350k real interrupts per second=2E > > The following historical context comes from jhb@: Originally, the > threshold > worked around incorrect routing of PCI INTx interrupts on single-CPU > systems > which would end up in a hard hang during boot=2E Since the threshold = was > added, our PCI interrupt routing was improved, most PCI interrupts use > edge-triggered MSI instead of level-triggered INTx, and typical system= s > have > multiple CPUs available to service interrupts=2E > > On the off chance that the threshold is useful in the future, it remai= ns > available as a tunable and sysctl=2E > > Reviewed by: jhb > Sponsored by: Dell EMC Isilon > Differential Revision: https://reviews=2Efreebsd=2Eorg/D20401 > > Modified: > head/sys/kern/kern_intr=2Ec > > Modified: head/sys/kern/kern_intr=2Ec > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- head/sys/kern/kern_intr=2Ec Fri May 24 22:30:40 2019 (r3482= 54) > +++ head/sys/kern/kern_intr=2Ec Fri May 24 22:33:14 2019 (r3482= 55) > @@ -91,7 +91,7 @@ struct proc *intrproc; > > static MALLOC_DEFINE(M_ITHREAD, "ithread", "Interrupt Threads"); > > -static int intr_storm_threshold =3D 1000; > +static int intr_storm_threshold =3D 0; > SYSCTL_INT(_hw, OID_AUTO, intr_storm_threshold, CTLFLAG_RWTUN, > &intr_storm_threshold, 0, > "Number of consecutive interrupts before storm protection is > enabled"); > > --=20 Pardon the typos and autocorrect, small keyboard in use=2E Cheers, Cy Schubert FreeBSD UNIX: Web: http://www=2EFreeBSD=2Eorg The need of the many outweighs the greed of the few=2E From owner-svn-src-head@freebsd.org Sat May 25 01:59:26 2019 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id E5D8115BA6FA; Sat, 25 May 2019 01:59:25 +0000 (UTC) (envelope-from cem@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 88A6388E1C; Sat, 25 May 2019 01:59:25 +0000 (UTC) (envelope-from cem@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 631192562D; Sat, 25 May 2019 01:59:25 +0000 (UTC) (envelope-from cem@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id x4P1xPHA077153; Sat, 25 May 2019 01:59:25 GMT (envelope-from cem@FreeBSD.org) Received: (from cem@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id x4P1xPiV077152; Sat, 25 May 2019 01:59:25 GMT (envelope-from cem@FreeBSD.org) Message-Id: <201905250159.x4P1xPiV077152@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: cem set sender to cem@FreeBSD.org using -f From: Conrad Meyer Date: Sat, 25 May 2019 01:59:25 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r348262 - head/sys/dev/virtio/pci X-SVN-Group: head X-SVN-Commit-Author: cem X-SVN-Commit-Paths: head/sys/dev/virtio/pci X-SVN-Commit-Revision: 348262 X-SVN-Commit-Repository: base MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Rspamd-Queue-Id: 88A6388E1C X-Spamd-Bar: -- Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-2.98 / 15.00]; local_wl_from(0.00)[FreeBSD.org]; NEURAL_HAM_MEDIUM(-1.00)[-0.998,0]; NEURAL_HAM_SHORT(-0.98)[-0.983,0]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; ASN(0.00)[asn:11403, ipnet:2610:1c1:1::/48, country:US] X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 25 May 2019 01:59:26 -0000 Author: cem Date: Sat May 25 01:59:24 2019 New Revision: 348262 URL: https://svnweb.freebsd.org/changeset/base/348262 Log: virtio_pci(4): Fix typo in read_ivar method Prior to this revision, vtpci's BUS_READ_IVAR method on VIRTIO_IVAR_SUBVENDOR accidentally returned the PCI subdevice. The typo seems to have been introduced with the original commit adding VIRTIO_IVAR_{{SUB,}DEVICE,{SUB,}VENDOR} to virtio_pci. The commit log and code strongly suggest that the ivar was intended to return the subvendor rather than the subdevice; it was likely just a copy/paste mistake. Go ahead and rectify that. Modified: head/sys/dev/virtio/pci/virtio_pci.c Modified: head/sys/dev/virtio/pci/virtio_pci.c ============================================================================== --- head/sys/dev/virtio/pci/virtio_pci.c Sat May 25 01:58:00 2019 (r348261) +++ head/sys/dev/virtio/pci/virtio_pci.c Sat May 25 01:59:24 2019 (r348262) @@ -408,7 +408,7 @@ vtpci_read_ivar(device_t dev, device_t child, int inde *result = pci_get_device(dev); break; case VIRTIO_IVAR_SUBVENDOR: - *result = pci_get_subdevice(dev); + *result = pci_get_subvendor(dev); break; default: return (ENOENT);