Page MenuHomeFreeBSD
Differential D39799

pxeboot: bugs in pxe.h
ClosedPublic
Actions

Authored by tsoome on Apr 25 2023, 12:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 17, 1:27 PM
Unknown Object (File)
Sat, Jan 17, 6:13 AM
Unknown Object (File)
Fri, Jan 16, 11:21 PM
Unknown Object (File)
Fri, Jan 16, 9:57 PM
Unknown Object (File)
Wed, Jan 7, 2:03 PM
Unknown Object (File)
Sat, Jan 3, 3:55 PM
Unknown Object (File)
Thu, Jan 1, 6:38 PM
Unknown Object (File)
Dec 12 2025, 3:34 AM
View All Files
Subscribers
dab
imp

Details

Reviewers
manu
imp
Commits
rG795497bf3d2c: pxeboot: bugs in pxe.h
Summary

SEGDESC_t needs to be PACKED
there is no status in t_PXENV_UNDI_MCAST_ADDRESS

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tsoome created this revision.Apr 25 2023, 12:13 PM
Herald added a reviewer: manu. · View Herald TranscriptApr 25 2023, 12:13 PM
Herald added subscribers: dab, imp. · View Herald Transcript
tsoome requested review of this revision.Apr 25 2023, 12:13 PM
Harbormaster completed remote builds in B51175: Diff 120997.Apr 25 2023, 12:13 PM
imp accepted this revision.Apr 25 2023, 10:42 PM

so how was all this working before?
or is SEGDESC_t not all that important?

This revision is now accepted and ready to land.Apr 25 2023, 10:42 PM
tsoome added a comment.Apr 25 2023, 10:43 PM
In D39799#906388, @imp wrote:

so how was all this working before?
or is SEGDESC_t not all that important?

We are not using those fields, nor support multicast.

Closed by commit rG795497bf3d2c: pxeboot: bugs in pxe.h (authored by tsoome). · Explain WhyApr 26 2023, 1:34 PM
This revision was automatically updated to reflect the committed changes.
tsoome added a commit: rG795497bf3d2c: pxeboot: bugs in pxe.h.

Revision Contents
Changeset List

PathSize
stand/
i386/
libi386/
3 lines

Diff 121078

View Options

stand/i386/libi386/pxe.h

Loading...