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
F161589592: D39799.diff
Sun, Jul 5, 3:55 AM
F161589568: D39799.diff
Sun, Jul 5, 3:54 AM
Unknown Object (File)
Tue, Jun 30, 7:39 AM
Unknown Object (File)
Sun, Jun 28, 7:47 PM
Unknown Object (File)
Sat, Jun 27, 2:45 AM
Unknown Object (File)
Thu, Jun 25, 9:54 PM
Unknown Object (File)
Tue, Jun 23, 9:11 PM
Unknown Object (File)
Tue, Jun 23, 8:43 PM
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...