arc patch does not work with this revision, probably because of this Phabricator issue: https://secure.phabricator.com/T9069
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Sep 4 2020
Remove ! characters from converted README
Convert DDP README from utf-16le/dos to utf-8/unix
In D26322#585095, @lwhsu wrote:Why is sys/contrib/dev/ice/README a binary file?
Sep 1 2020
In D26276#584027, @mhorne wrote:Hi, adding some Intel folks who have appeared on recent ixl(4) reviews/submissions.
I am attempting to upstream this patch from NetApp, who wrote it originally.
It seems that updates to these definitions usually come from upstream as part of
larger ixl driver updates, so forgive me if this is not the usual process.
Aug 25 2020
Any updates on this?
Aug 22 2020
New poudriere testport log:
New portlint output:
WARN: Makefile: for new port, make $FreeBSD$ tag in comment section empty, to make SVN happy. 0 fatal errors and 1 warning found.
In D26118#580232, @mat wrote:In D26118#579726, @erj wrote:Output from portlint -AC:
WARN: Makefile: for new port, make $FreeBSD$ tag in comment section empty, to make SVN happy. WARN: Makefile: "IGNORE_DragonFly" has to appear earlier. WARN: Makefile: "ONLY_FOR_ARCHS" has to appear earlier. WARN: Makefile: "ONLY_FOR_ARCHS_REASON" has to appear earlier. 0 fatal errors and 4 warnings found.Please do fix those warnings. The reference for how to write a Makefile is available there https://www.freebsd.org/doc/en/books/porters-handbook/porting-order.html
Edit Makefile to make portlint happy
Aug 21 2020
I've attached the poudriere testport log:
Aug 18 2020
portlint -AC output:
WARN: Makefile: for new port, make $FreeBSD$ tag in comment section empty, to make SVN happy. WARN: Makefile: unless this is a master port, PORTNAME has to be set by "=", not by "?=". WARN: Makefile: unless this is a master port, PORTVERSION has to be set by "=", not by "?=". WARN: Makefile: new ports should not set PORTREVISION. WARN: Makefile: unless this is a master port, MAINTAINER has to be set by "=", not by "?=". WARN: Makefile: unless this is a master port, COMMENT has to be set by "=", not by "?=". WARN: Makefile: "ONLY_FOR_ARCHS" has to appear earlier. WARN: Makefile: "ONLY_FOR_ARCHS_REASON" has to appear earlier. 0 fatal errors and 8 warnings found.
Aug 10 2020
@jacob.e.keller_intel.com do you maybe have any thoughts on this, too?
The latter nvm related functions can be removed immediately; these have been removed in a newer version of the driver.
Aug 4 2020
Jul 30 2020
Jul 29 2020
Jun 26 2020
Is someone else going to take over? Is this still something worthwhile to pursue? It seems like it based on the latest comments.
Jun 25 2020
How did you find this? Do you have something automated that checks for this?
Jun 12 2020
Jun 11 2020
Jun 10 2020
I committed this in rS361992.
Jun 9 2020
This description looks better; thanks.
Jun 4 2020
Before I commit this, I'd like to have a better description of the changes that are going into this patch; I see there are changes around Recovery Mode support, but I'd like things like significant bugs or feature additions to be called out since this is pretty large.
May 28 2020
This was committed in rS361541.
May 26 2020
May 22 2020
Any other comments before I commit this?
May 21 2020
- ice: Update MAINTAINERS file
- ice: Remove strange characters in ice_ddp README
May 14 2020
- ice: Add entries to arm64 NOTES file
- ice: Add support for building on arm64 targets
May 13 2020
In D21959#546203, @kbowling wrote:Can we enable this in an unsupported state for other 64-bit archs (arm64. ppc64)? Just would like to see the files entry correct for such usage.
May 12 2020
In D21959#546203, @kbowling wrote:Can we enable this in an unsupported state for other 64-bit archs (arm64. ppc64)? Just would like to see the files entry correct for such usage.
May 11 2020
- ice-ddp: Use "SRCTOP" instead of "S" in module Makefile
May 6 2020
- Add em_if_needs_restart to igb's iflib device method list
May 1 2020
Apr 27 2020
Apr 8 2020
- Use iflib_init_locked()
Fix incorrect type usage
- Follow Drew's suggestion and create an iflib method for this
Mar 30 2020
This doesn't apply anymore, at least in part because of rS359438
Mar 27 2020
I'll add @adrian to see if I'm using the firmware(9) functionality correctly enough; it's similar to how it's used in the Wi-Fi drivers I think. Add other reviewers if you think it might be helpful.
- Add ddp to kernel and build process
- ice: Use fancier ice module Makefile from OOT component
- Move ice files.amd64 entries to appropriate place
- ice: Update description in NOTES
Mar 25 2020
Mar 20 2020
- Remove extraneous line breaks from files.amd64
- ice: Fix spelling mistake caught by arcanist
- ice: Add missing includes to ice_iflib.h
- ice: Correct filename used in Makefile
- ice: Add files.amd64 entries for ice
- ice: Add "device ice" as a kernel config option; add it to GENERIC
- ice: Add ice module to sys/modules Makefile
- ice: Remove SR-IOV files/option from Makefile
- ice: Update ice driver code to a current version.
Mar 12 2020
I'm generally okay with this, but with the caveat that we can't test to make sure any future changes we make will work on anything other than amd64.
Mar 5 2020
The driver is only intended to work on amd64; I can accept PowerPC64 but we don't want to support this driver on 32-bit architectures.
Thanks for fixing the issue with IXL_DEBUG! The CPU_TO_LE16 changes look acceptable to me, but we can't guarantee we'll be vigilant about adding them to new code in the future since we only officially support amd64 for ixl.
Mar 3 2020
So it sounds like you should keep the patch as-is, but then remove the limit sysctls and the arbitrary limit in a follow-on patch, possibly intended to be MFC'd in to 12.2?
One driver that is subject to the above scenario is the ixl driver.
I think this looks okay, but I don't think it'll apply to the Intel drivers, right?
Feb 12 2020
Is there a reason that these syctls would need "CTLFLAG_NEEDSGIANT", or does this patch default to that flag because it's a safe default if the contents of the sysctl are unknown? I'm aware of the previous review that added these flags, but I didn't get to look at it too closely.
Jan 13 2020
Jan 10 2020
Jan 3 2020
Can this get regenerated? It no longer applies cleanly.