Page MenuHomeFreeBSD
Differential D35153

kdump: Decode sigset_t.
ClosedPublic
Actions

Authored by dchagin on May 9 2022, 6:58 PM.
Tags
None
Referenced Files
F157176265: D35153.diff
Tue, May 19, 1:26 AM
Unknown Object (File)
Sat, May 16, 12:24 AM
Unknown Object (File)
Sat, May 16, 12:24 AM
Unknown Object (File)
Sat, May 16, 12:24 AM
Unknown Object (File)
Sat, May 16, 12:24 AM
Unknown Object (File)
Sat, May 16, 12:24 AM
Unknown Object (File)
Sat, May 16, 12:23 AM
Unknown Object (File)
Sat, May 9, 8:53 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 45774
Build 42662: 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
217
218
221
224

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
191

name and mask should be declared const.

212

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
usr.bin/
kdump/
1 line
5 lines
47 lines

Diff 106483

View Options

usr.bin/kdump/kdump.h

Loading...
View Options

usr.bin/kdump/kdump.c

Loading...
View Options

usr.bin/kdump/linux.c

Loading...