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)
Fri, Dec 20, 4:42 AM
Unknown Object (File)
Wed, Dec 11, 10:45 PM
Unknown Object (File)
Nov 6 2024, 3:48 PM
Unknown Object (File)
Nov 5 2024, 3:24 AM
Unknown Object (File)
Nov 5 2024, 3:24 AM
Unknown Object (File)
Nov 5 2024, 3:13 AM
Unknown Object (File)
Sep 27 2024, 8:51 AM
Unknown Object (File)
Sep 24 2024, 9:47 AM
View All 44 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 Passed
Unit
No Test Coverage
Build Status
Buildable 51175
Build 48066: arc lint + arc unit

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 120997

View Options

stand/i386/libi386/pxe.h

Loading...