as provided by the Intel document 346446-009 AKA FRED 8.0.
Diff Detail
- Repository
- rG FreeBSD src repository
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
| sys/amd64/amd64/trap.c | ||
|---|---|---|
| 944 | No need anymore for the mask of tf_cs. | |
| sys/x86/include/frame.h | ||
| 148 | The field prefix should be tf_. | |
| 152–157 | Should we apply some explicit masking in places where tf_ss is assigned by software? e.g. in set_mcontext(), we have tp->tf_ss = mcp->mc_ss, so there is implicit truncation. | |
| 154 | The numbering of the evinfo fields is a bit weird. I see why you did it this way, it follows the order that the fields are described in the FRED documentation. Probably not worth changing. | |
| sys/x86/include/frame.h | ||
|---|---|---|
| 152–157 | I do not object, but I do not see a purpose for explicit cast or masking. | |
| sys/x86/include/frame.h | ||
|---|---|---|
| 152–157 | Some checks would be good. I'm not sure if there is any chance that they would break compatibility somehow. | |