Page MenuHomeFreeBSD
Differential D25152

sys/riscv: Remove debug printfs
ClosedPublic
Actions

Authored by arichardson on Jun 5 2020, 5:19 PM.
Tags
None
Referenced Files
F116257407: D25152.id72815.diff
Sun, May 4, 11:16 AM
Unknown Object (File)
Wed, Apr 23, 2:02 AM
Unknown Object (File)
Tue, Apr 22, 12:35 PM
Unknown Object (File)
Tue, Apr 15, 12:22 AM
Unknown Object (File)
Mar 22 2025, 12:18 AM
Unknown Object (File)
Mar 18 2025, 5:16 AM
Unknown Object (File)
Mar 15 2025, 10:48 PM
Unknown Object (File)
Mar 4 2025, 1:28 AM
View All 43 Files
Subscribers
imp
jrtc27
mhorne

Details

Reviewers
br
mhorne
Group Reviewers
riscv
Commits
rS361904: sys/riscv: Remove debug printfs
Summary

They are only visible with EARLY_PRINTF so don't show up by default.

Diff Detail

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

Event Timeline

arichardson requested review of this revision.Jun 5 2020, 5:19 PM
arichardson created this revision.
Harbormaster completed remote builds in B31513: Diff 72724.Jun 5 2020, 5:19 PM
arichardson added inline comments.Jun 5 2020, 5:19 PM
sys/riscv/riscv/pmap.c
565 ↗(On Diff #72724)

Should this one also be deleted?

jrtc27 added a subscriber: jrtc27.Jun 5 2020, 5:25 PM

FWIW, this series of three printf's was copied directly from arm64's pmap_bootstrap.

mhorne accepted this revision.Jun 5 2020, 5:42 PM
mhorne added a subscriber: mhorne.
mhorne added inline comments.
sys/riscv/riscv/pmap.c
565 ↗(On Diff #72724)

I'd vote for it to stay, I've used it for debugging before.

This revision is now accepted and ready to land.Jun 5 2020, 5:42 PM
Closed by commit rS361904: sys/riscv: Remove debug printfs (authored by arichardson). · Explain WhyJun 8 2020, 8:52 AM
This revision was automatically updated to reflect the committed changes.
arichardson added a commit: rS361904: sys/riscv: Remove debug printfs.
Herald added a subscriber: imp. · View Herald TranscriptJun 8 2020, 8:52 AM

Revision Contents
Changeset List

PathSize
head/
sys/
riscv/
riscv/
2 lines

Diff 72815

View Options

head/sys/riscv/riscv/pmap.c

Loading...