<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src/sys/modules/qlnx, branch main</title>
<subtitle>FreeBSD source tree</subtitle>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/'/>
<entry>
<title>qlnxev: Remove now unneeded include for opt_inet.h</title>
<updated>2026-02-06T17:54:58+00:00</updated>
<author>
<name>Zhenlei Huang</name>
<email>zlei@FreeBSD.org</email>
</author>
<published>2026-02-06T17:52:54+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=ec7950fe42344900567cb72c83845ea4dc5a7114'/>
<id>ec7950fe42344900567cb72c83845ea4dc5a7114</id>
<content type='text'>
Since the change [1], this is not required anymore.

This change partially reverts commit 8a847947153e.

[1] 4012b63889e4 qlnxe: Let ether_ioctl() handle SIOCSIFADDR ioctl

Reviewed by:	kbowling
MFC after:	5 days
Differential Revision:	https://reviews.freebsd.org/D54889
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Since the change [1], this is not required anymore.

This change partially reverts commit 8a847947153e.

[1] 4012b63889e4 qlnxe: Let ether_ioctl() handle SIOCSIFADDR ioctl

Reviewed by:	kbowling
MFC after:	5 days
Differential Revision:	https://reviews.freebsd.org/D54889
</pre>
</div>
</content>
</entry>
<entry>
<title>qlnxe: Let ether_ioctl() handle SIOCSIFADDR ioctl</title>
<updated>2026-02-06T17:54:15+00:00</updated>
<author>
<name>Zhenlei Huang</name>
<email>zlei@FreeBSD.org</email>
</author>
<published>2026-02-06T17:52:54+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=4012b63889e40bb877bc0e4c8da1792bce472c08'/>
<id>4012b63889e40bb877bc0e4c8da1792bce472c08</id>
<content type='text'>
Since the change [1], the init routine qlnx_init() works as intended.
Let ether_ioctl() handle SIOCSIFADDR to simplify the code.

Combined with the change [1], this shall be a better fix for PR 287445.

[1] c10e6bc0f007 qlnxe: Avoid reinitializing the interface when it is already initialized

PR:		287445
Reviewed by:	kbowling
MFC after:	5 days
Differential Revision:	https://reviews.freebsd.org/D54888
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Since the change [1], the init routine qlnx_init() works as intended.
Let ether_ioctl() handle SIOCSIFADDR to simplify the code.

Combined with the change [1], this shall be a better fix for PR 287445.

[1] c10e6bc0f007 qlnxe: Avoid reinitializing the interface when it is already initialized

PR:		287445
Reviewed by:	kbowling
MFC after:	5 days
Differential Revision:	https://reviews.freebsd.org/D54888
</pre>
</div>
</content>
</entry>
<entry>
<title>kern: Remove needless kern.opts.mk</title>
<updated>2025-10-10T00:12:38+00:00</updated>
<author>
<name>Warner Losh</name>
<email>imp@FreeBSD.org</email>
</author>
<published>2025-10-08T04:17:18+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=8e985774117d49eb968789aba8550c2a92992a9a'/>
<id>8e985774117d49eb968789aba8550c2a92992a9a</id>
<content type='text'>
We don't need kern.opts.mk in any of these places. None of these
Makefiles reference any MK_ options. Some don't even need SYSDIR,
but leave that defined in the ones that do.

Sponsored by:		Netflix
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
We don't need kern.opts.mk in any of these places. None of these
Makefiles reference any MK_ options. Some don't even need SYSDIR,
but leave that defined in the ones that do.

Sponsored by:		Netflix
</pre>
</div>
</content>
</entry>
<entry>
<title>sys/modules: fix standalone build for multiple modules</title>
<updated>2025-08-23T04:58:40+00:00</updated>
<author>
<name>ShengYi Hung</name>
<email>aokblast@FreeBSD.org</email>
</author>
<published>2025-08-21T08:27:06+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=8a847947153e60d812852e5c02de93997b2d3038'/>
<id>8a847947153e60d812852e5c02de93997b2d3038</id>
<content type='text'>
Multiple Makefile miss opt_*.h and *_if.h header file. We fix it by
running make in sys/modules to build all modules.

Approved by:    lwhsu (mentor), markj (mentor)
Sponsored by:   The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D52062
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Multiple Makefile miss opt_*.h and *_if.h header file. We fix it by
running make in sys/modules to build all modules.

Approved by:    lwhsu (mentor), markj (mentor)
Sponsored by:   The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D52062
</pre>
</div>
</content>
</entry>
<entry>
<title>qlnxe: Fix build with INET</title>
<updated>2025-07-17T04:01:16+00:00</updated>
<author>
<name>Zhenlei Huang</name>
<email>zlei@FreeBSD.org</email>
</author>
<published>2025-07-17T04:01:16+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=10d9ec31bf28949221280c9ddc0816464407a43f'/>
<id>10d9ec31bf28949221280c9ddc0816464407a43f</id>
<content type='text'>
opt_inet.h is missing from 5684c8783b64, thus actually `#ifdef INET`
does not work.

PR:		287445
See also:	https://redmine.pfsense.org/issues/16248
Reviewed by:	kbowling
Tested by:	leper &lt;leper4@protonmail.com&gt;
Fixes:		5684c8783b64 qlnxe: Add missing #ifdef INET to fix amd64 LINT-NOINET/NOIP builds
MFC after:	3 days
Differential Revision:	https://reviews.freebsd.org/D51295
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
opt_inet.h is missing from 5684c8783b64, thus actually `#ifdef INET`
does not work.

PR:		287445
See also:	https://redmine.pfsense.org/issues/16248
Reviewed by:	kbowling
Tested by:	leper &lt;leper4@protonmail.com&gt;
Fixes:		5684c8783b64 qlnxe: Add missing #ifdef INET to fix amd64 LINT-NOINET/NOIP builds
MFC after:	3 days
Differential Revision:	https://reviews.freebsd.org/D51295
</pre>
</div>
</content>
</entry>
<entry>
<title>sys: Remove $FreeBSD$: one-line sh pattern</title>
<updated>2023-08-16T17:54:58+00:00</updated>
<author>
<name>Warner Losh</name>
<email>imp@FreeBSD.org</email>
</author>
<published>2023-08-16T17:54:58+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=031beb4e239bfce798af17f5fe8dba8bcaf13d99'/>
<id>031beb4e239bfce798af17f5fe8dba8bcaf13d99</id>
<content type='text'>
Remove /^\s*#[#!]?\s*\$FreeBSD\$.*$\n/
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Remove /^\s*#[#!]?\s*\$FreeBSD\$.*$\n/
</pre>
</div>
</content>
</entry>
<entry>
<title>qlnxe: Fix unused but set variable warning</title>
<updated>2023-07-01T17:06:58+00:00</updated>
<author>
<name>Mark O'Donovan</name>
<email>shiftee@posteo.net</email>
</author>
<published>2023-07-01T17:06:51+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=dc58675cfb18b1fcde4cdad95eae546c0d5f35cb'/>
<id>dc58675cfb18b1fcde4cdad95eae546c0d5f35cb</id>
<content type='text'>
Signed-off-by: Mark O'Donovan &lt;shiftee@posteo.net&gt;
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/779
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Mark O'Donovan &lt;shiftee@posteo.net&gt;
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/779
</pre>
</div>
</content>
</entry>
<entry>
<title>qlnxe: Remove #ifdef ECORE_PACKAGE</title>
<updated>2023-07-01T17:06:57+00:00</updated>
<author>
<name>Mark O'Donovan</name>
<email>shiftee@posteo.net</email>
</author>
<published>2023-07-01T17:06:50+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=b75c387a668e3c508d1cdc6e2aa7135aff1b7311'/>
<id>b75c387a668e3c508d1cdc6e2aa7135aff1b7311</id>
<content type='text'>
I do not know why this is here but it blocks compilation.
Removing it makes the builtin option the same as the module build

Signed-off-by: Mark O'Donovan &lt;shiftee@posteo.net&gt;
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/779
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
I do not know why this is here but it blocks compilation.
Removing it makes the builtin option the same as the module build

Signed-off-by: Mark O'Donovan &lt;shiftee@posteo.net&gt;
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/779
</pre>
</div>
</content>
</entry>
<entry>
<title>qlnxe: Remove -Wno-case-qual from module build</title>
<updated>2023-07-01T17:06:55+00:00</updated>
<author>
<name>Mark O'Donovan</name>
<email>shiftee@posteo.net</email>
</author>
<published>2023-07-01T17:06:50+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=134b09369960a84cf698a639f7f20be3e10966b6'/>
<id>134b09369960a84cf698a639f7f20be3e10966b6</id>
<content type='text'>
Signed-off-by: Mark O'Donovan &lt;shiftee@posteo.net&gt;
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/779
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Mark O'Donovan &lt;shiftee@posteo.net&gt;
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/779
</pre>
</div>
</content>
</entry>
<entry>
<title>LinuxKPI: define LINUXKPI_INCLUDES for module builds as well</title>
<updated>2022-09-26T19:34:01+00:00</updated>
<author>
<name>Bjoern A. Zeeb</name>
<email>bz@FreeBSD.org</email>
</author>
<published>2022-09-23T21:18:45+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=514fb3872166a361faa81d89909f65913e90b1b2'/>
<id>514fb3872166a361faa81d89909f65913e90b1b2</id>
<content type='text'>
While for in-kernel we already have LINUXKPI_INCLUDES in kern.pre.mk
for kmod builds we've not had a common define to use leading to various
spellings of include paths.

In order for the include list to be expanded more easily in the future,
e.g., adding the "dummy" includes (for all) and to harmonize code,
duplicate LINUXKPI_INCLUDES to kmod.mk and use it for all module Makefiles.

MFC after:	1 week
Reviewed by:	hselasky
Differential Revision: https://reviews.freebsd.org/D36683
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
While for in-kernel we already have LINUXKPI_INCLUDES in kern.pre.mk
for kmod builds we've not had a common define to use leading to various
spellings of include paths.

In order for the include list to be expanded more easily in the future,
e.g., adding the "dummy" includes (for all) and to harmonize code,
duplicate LINUXKPI_INCLUDES to kmod.mk and use it for all module Makefiles.

MFC after:	1 week
Reviewed by:	hselasky
Differential Revision: https://reviews.freebsd.org/D36683
</pre>
</div>
</content>
</entry>
</feed>
