<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src/sys/dev/ipmi, branch main</title>
<subtitle>FreeBSD source tree</subtitle>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/'/>
<entry>
<title>chore: replace {0, 0} with {DEV,KOBJ}METHOD_END</title>
<updated>2026-02-25T05:59:34+00:00</updated>
<author>
<name>Enji Cooper</name>
<email>ngie@FreeBSD.org</email>
</author>
<published>2026-02-21T06:14:46+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=0fc6c3f731a2cca3120798806c330a3081c9424b'/>
<id>0fc6c3f731a2cca3120798806c330a3081c9424b</id>
<content type='text'>
Both of the aforementioned macros have been present in FreeBSD
for well over a decade: 2009 for `KOBJMETHOD_END`; 2011 for
`DEVMETHOD_END`.

Adapt all hardcoded references of `{0, 0}` with `DEVMETHOD_END`
and `KOBJMETHOD_END` as appropriate. This helps ensure that
future adaptations to drivers following patterns documented
in driver(9) can be made more easily/without issue.

MFC after:	1 week
Differential Revision:	 https://reviews.freebsd.org/D55414
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Both of the aforementioned macros have been present in FreeBSD
for well over a decade: 2009 for `KOBJMETHOD_END`; 2011 for
`DEVMETHOD_END`.

Adapt all hardcoded references of `{0, 0}` with `DEVMETHOD_END`
and `KOBJMETHOD_END` as appropriate. This helps ensure that
future adaptations to drivers following patterns documented
in driver(9) can be made more easily/without issue.

MFC after:	1 week
Differential Revision:	 https://reviews.freebsd.org/D55414
</pre>
</div>
</content>
</entry>
<entry>
<title>ipmi: Migrate to use macro LINUX_IOCTL_SET to register linux ioctl handler</title>
<updated>2025-10-20T04:28:42+00:00</updated>
<author>
<name>Zhenlei Huang</name>
<email>zlei@FreeBSD.org</email>
</author>
<published>2025-10-20T04:28:42+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=cb0116af7ac794238b0fa569f2722607924fcad6'/>
<id>cb0116af7ac794238b0fa569f2722607924fcad6</id>
<content type='text'>
Reviewed by:	markj
MFC after:	2 weeks
Differential Revision:	https://reviews.freebsd.org/D53158
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Reviewed by:	markj
MFC after:	2 weeks
Differential Revision:	https://reviews.freebsd.org/D53158
</pre>
</div>
</content>
</entry>
<entry>
<title>newbus: replace leftover device unit wildcards</title>
<updated>2025-06-21T02:57:45+00:00</updated>
<author>
<name>Ahmad Khalifa</name>
<email>vexeduxr@FreeBSD.org</email>
</author>
<published>2025-06-11T00:25:08+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=b670c9bafc0e31c7609969bf374b2e80bdc00211'/>
<id>b670c9bafc0e31c7609969bf374b2e80bdc00211</id>
<content type='text'>
Reviewed by:	imp, jhb
Approved by:	imp (mentor)
Differential Revision:	https://reviews.freebsd.org/D50913
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Reviewed by:	imp, jhb
Approved by:	imp (mentor)
Differential Revision:	https://reviews.freebsd.org/D50913
</pre>
</div>
</content>
</entry>
<entry>
<title>efi: Use EDK2 EFI_GUID instead of sturct uuid</title>
<updated>2025-05-01T17:53:28+00:00</updated>
<author>
<name>Warner Losh</name>
<email>imp@FreeBSD.org</email>
</author>
<published>2025-05-01T17:53:28+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=6366114c716f9dca099a4090269a700b94a6173f'/>
<id>6366114c716f9dca099a4090269a700b94a6173f</id>
<content type='text'>
So, while struct uuid and EFI_GUID are quite similar (they are the same
size, they have the same binary representation, etc), they aren not
identical. Speciifcally, the UUID and GUID defines are slightly
different and can't be swapped one for the other. Start to use EFI_GUID
for all EFI tables. This will allow increased sharing with the boot
laoder that already uses EDK2 variants as well as using the possibility
to use the EDK2 headers we've already imported more easily.

Sponsored by:		Netflix
Differential Revision:	https://reviews.freebsd.org/D50037
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
So, while struct uuid and EFI_GUID are quite similar (they are the same
size, they have the same binary representation, etc), they aren not
identical. Speciifcally, the UUID and GUID defines are slightly
different and can't be swapped one for the other. Start to use EFI_GUID
for all EFI tables. This will allow increased sharing with the boot
laoder that already uses EDK2 variants as well as using the possibility
to use the EDK2 headers we've already imported more easily.

Sponsored by:		Netflix
Differential Revision:	https://reviews.freebsd.org/D50037
</pre>
</div>
</content>
</entry>
<entry>
<title>ipmi: fix runtime on powerpc64le</title>
<updated>2025-03-27T11:57:28+00:00</updated>
<author>
<name>Piotr Kubaj</name>
<email>pkubaj@FreeBSD.org</email>
</author>
<published>2025-03-27T11:55:48+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=2e16618fe789f110bd8d297b1c65b166fa60c2fe'/>
<id>2e16618fe789f110bd8d297b1c65b166fa60c2fe</id>
<content type='text'>
Summary:
OPAL runs in big-endian mode, so we need to byteswap msg_len on little-endian.

MFC after:	1 week

Test Plan: ipmitool lan print

Reviewers: #powerpc

Subscribers: imp

Differential Revision: https://reviews.freebsd.org/D49530
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Summary:
OPAL runs in big-endian mode, so we need to byteswap msg_len on little-endian.

MFC after:	1 week

Test Plan: ipmitool lan print

Reviewers: #powerpc

Subscribers: imp

Differential Revision: https://reviews.freebsd.org/D49530
</pre>
</div>
</content>
</entry>
<entry>
<title>ipmi: fix powerpc build</title>
<updated>2024-10-18T20:30:44+00:00</updated>
<author>
<name>Gleb Smirnoff</name>
<email>glebius@FreeBSD.org</email>
</author>
<published>2024-10-18T20:30:02+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=b75e32a466dbab7a6729ff7a6472c6f9c9207491'/>
<id>b75e32a466dbab7a6729ff7a6472c6f9c9207491</id>
<content type='text'>
Fixes:	366d6a424e1faad9c151c5794978e218a79fbed8
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Fixes:	366d6a424e1faad9c151c5794978e218a79fbed8
</pre>
</div>
</content>
</entry>
<entry>
<title>ipmi: remove timeout from the ipmi_driver_request method</title>
<updated>2024-10-18T19:30:09+00:00</updated>
<author>
<name>Gleb Smirnoff</name>
<email>glebius@FreeBSD.org</email>
</author>
<published>2024-10-18T19:30:09+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=366d6a424e1faad9c151c5794978e218a79fbed8'/>
<id>366d6a424e1faad9c151c5794978e218a79fbed8</id>
<content type='text'>
Driver requests are done with stack allocated request.  The request is
put on the tailq and then we msleep(9) until kernel process processes it.
If we timeout from this sleep, the kernel process may still read the
request from our stack, which may already be reused by some other code.

Make this sleep unbound and rely on the kernel process that does all its
I/O with timouts and will eventually wake us up.

Reviewed by:	jhb
Differential Revision:	https://reviews.freebsd.org/D47179
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Driver requests are done with stack allocated request.  The request is
put on the tailq and then we msleep(9) until kernel process processes it.
If we timeout from this sleep, the kernel process may still read the
request from our stack, which may already be reused by some other code.

Make this sleep unbound and rely on the kernel process that does all its
I/O with timouts and will eventually wake us up.

Reviewed by:	jhb
Differential Revision:	https://reviews.freebsd.org/D47179
</pre>
</div>
</content>
</entry>
<entry>
<title>smbios: Add length sanity checking</title>
<updated>2024-07-25T05:09:57+00:00</updated>
<author>
<name>Warner Losh</name>
<email>imp@FreeBSD.org</email>
</author>
<published>2024-07-25T05:02:27+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=06326613afebc645433c6bf8a2249cf978db9e71'/>
<id>06326613afebc645433c6bf8a2249cf978db9e71</id>
<content type='text'>
D28743 was commited, reverted and then f689cb23b2782 landed before it
was recommitted. However, D28743 included an extra length check. Redo
that functionality so we check both the number of entries as well as the
length checks for wacky data.

Sponsored by:		Netflix
Reviewed by:		gallatin
Differential Revision:	https://reviews.freebsd.org/D45763
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
D28743 was commited, reverted and then f689cb23b2782 landed before it
was recommitted. However, D28743 included an extra length check. Redo
that functionality so we check both the number of entries as well as the
length checks for wacky data.

Sponsored by:		Netflix
Reviewed by:		gallatin
Differential Revision:	https://reviews.freebsd.org/D45763
</pre>
</div>
</content>
</entry>
<entry>
<title>ipmi_smbios: Only search for the BIOS signature on x86</title>
<updated>2024-07-25T05:09:57+00:00</updated>
<author>
<name>Warner Losh</name>
<email>imp@FreeBSD.org</email>
</author>
<published>2024-07-25T05:09:21+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=ce41bee0b4b876a55a94ce7cfa77421e7b274a94'/>
<id>ce41bee0b4b876a55a94ce7cfa77421e7b274a94</id>
<content type='text'>
Sponsored by:		Netflix
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Sponsored by:		Netflix
</pre>
</div>
</content>
</entry>
<entry>
<title>newbus: replace -1 in BUS_ADD_CHILD(...-1) with DEVICE_UNIT_ANY</title>
<updated>2024-07-25T04:23:02+00:00</updated>
<author>
<name>Warner Losh</name>
<email>imp@FreeBSD.org</email>
</author>
<published>2024-07-25T04:23:02+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=a05a680469a7ac77b195021fed74e3aa58152dd7'/>
<id>a05a680469a7ac77b195021fed74e3aa58152dd7</id>
<content type='text'>
Sponsored by:		Netflix
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Sponsored by:		Netflix
</pre>
</div>
</content>
</entry>
</feed>
