Page MenuHomeFreeBSD
Differential D35153

kdump: Decode sigset_t.
ClosedPublic
Actions

Authored by dchagin on May 9 2022, 6:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 1, 1:41 PM
Unknown Object (File)
Fri, Oct 31, 5:44 AM
Unknown Object (File)
Fri, Oct 31, 3:41 AM
Unknown Object (File)
Wed, Oct 29, 10:42 PM
Unknown Object (File)
Sun, Oct 26, 4:26 PM
Unknown Object (File)
Thu, Oct 23, 11:26 PM
Unknown Object (File)
Sat, Oct 18, 8:42 AM
Unknown Object (File)
Fri, Oct 17, 6:19 PM
View All Files
Subscribers
imp

Details

Reviewers
jhb
markj
emaste
Commits
rG2068608ebdbf: kdump: Decode Linux l_sigset_t.
rG3606a213bfa4: kdump: Decode Linux l_sigset_t.
Summary

I'll add consumers later.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 45503
Build 42391: arc lint + arc unit

Event Timeline

dchagin created this revision.May 9 2022, 6:58 PM
Herald added a subscriber: imp. · View Herald TranscriptMay 9 2022, 6:58 PM
dchagin requested review of this revision.May 9 2022, 6:58 PM
Harbormaster completed remote builds in B45503: Diff 105791.May 9 2022, 6:58 PM
dchagin edited the summary of this revision. (Show Details)May 9 2022, 7:12 PM
dchagin added a reviewer: jhb.
dchagin changed the repository for this revision from rS FreeBSD src repository - subversion to rG FreeBSD src repository.
dchagin updated this revision to Diff 106483.May 30 2022, 5:18 PM
dchagin edited the summary of this revision. (Show Details)

move to linux.c

Harbormaster completed remote builds in B45774: Diff 106483.May 30 2022, 5:18 PM
dchagin added reviewers: email_luiz.eng.br, markj.May 30 2022, 5:50 PM
dchagin changed the repository for this revision from rS FreeBSD src repository - subversion to rG FreeBSD src repository.
dchagin added parent revisions: D35356: kdump: For future use extract common code to a separate files, D35216: kdump: Decode small amount of Linux syscalls.
dchagin edited reviewers, added: emaste; removed: email_luiz.eng.br.May 30 2022, 5:52 PM
markj added inline comments.May 31 2022, 2:56 PM
usr.bin/kdump/linux.c
191 ↗(On Diff #106483)
217 ↗(On Diff #106483)
218 ↗(On Diff #106483)
221 ↗(On Diff #106483)
224 ↗(On Diff #106483)

What's the point of making a copy of mask? ktrsigset() doesn't modify it as far as I can see...

dchagin updated this revision to Diff 106617.Jun 3 2022, 8:14 PM

yeah, copy-pasto fixed, thank you!

Harbormaster completed remote builds in B45838: Diff 106617.Jun 3 2022, 8:14 PM
markj added inline comments.Jun 3 2022, 8:47 PM
usr.bin/kdump/linux.c
194 ↗(On Diff #106617)

name and mask should be declared const.

215 ↗(On Diff #106617)

name and data should be declared const.

dchagin updated this revision to Diff 106625.Jun 3 2022, 10:27 PM

fixed, thanks, Mark

Harbormaster completed remote builds in B45846: Diff 106625.Jun 3 2022, 10:27 PM
markj accepted this revision.Jun 3 2022, 10:28 PM
This revision is now accepted and ready to land.Jun 3 2022, 10:28 PM
Closed by commit rG3606a213bfa4: kdump: Decode Linux l_sigset_t. (authored by dchagin). · Explain WhyJun 22 2022, 11:42 AM
This revision was automatically updated to reflect the committed changes.
dchagin added a commit: rG3606a213bfa4: kdump: Decode Linux l_sigset_t..
dchagin added a commit: rG2068608ebdbf: kdump: Decode Linux l_sigset_t..Jul 6 2022, 11:06 AM

Revision Contents
Changeset List

PathSize
sys/
sys/
2 lines
usr.bin/
kdump/
35 lines

Diff 105791

View Options

sys/sys/ktrace.h

Loading...
View Options

usr.bin/kdump/kdump.c

Loading...