Page MenuHomeFreeBSD

Remove yet another useless assignment
ClosedPublic

Authored by trasz on Oct 6 2020, 5:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jan 26, 6:12 PM
Unknown Object (File)
Fri, Jan 24, 7:17 PM
Unknown Object (File)
Tue, Jan 21, 9:15 AM
Unknown Object (File)
Dec 10 2024, 7:16 AM
Unknown Object (File)
Dec 5 2024, 6:25 PM
Unknown Object (File)
Oct 25 2024, 1:55 PM
Unknown Object (File)
Sep 24 2024, 11:11 PM
Unknown Object (File)
Sep 22 2024, 6:49 AM
Subscribers

Details

Summary

Remove yet another useless assignment, adding a KASSERT just in case.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

trasz requested review of this revision.Oct 6 2020, 5:56 PM
This revision is now accepted and ready to land.Oct 7 2020, 12:20 PM
arichardson added inline comments.
sys/riscv/riscv/trap.c
317 ↗(On Diff #77970)

Since frame is already stored to td->td_frame, we could drop this argument? And also the mv a0, sp in cpu_exception_handler_user ?
And then just add a struct trapframe *frame = td->td_frame and drop the assert?

sys/riscv/riscv/trap.c
317 ↗(On Diff #77970)

That's the plan. I'm just doing this one step at a time, to make it easier to bisect, should it become necessary.