Page MenuHomeFreeBSD
Differential D26816

Fix linprocfs(4) /proc/self/mem semantics
ClosedPublic
Actions

Authored by trasz on Oct 16 2020, 4:04 PM.
Tags
None
Referenced Files
F157999840: D26816.id78504.diff
Wed, May 27, 8:34 AM
F157999522: D26816.id78504.diff
Wed, May 27, 8:29 AM
F157991364: D26816.id78320.diff
Wed, May 27, 6:12 AM
F157991036: D26816.id78320.diff
Wed, May 27, 6:07 AM
Unknown Object (File)
Tue, May 26, 12:29 PM
Unknown Object (File)
Tue, May 26, 12:24 PM
Unknown Object (File)
Sat, May 9, 11:31 AM
Unknown Object (File)
Thu, May 7, 10:13 AM
View All Files
Subscribers
imp

Details

Reviewers
kib
Group Reviewers
Linux Emulation
Commits
rS366900: Fix linprocfs(4) /proc/self/mem semantics to more closely match Linux.
Summary

Fix linprocfs(4) /proc/self/mem semantics to more closely match Linux.
Steam's Anti-Cheat seems to use it.

PR: 248223

Diff Detail

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

Event Timeline

trasz created this revision.Oct 16 2020, 4:04 PM
Herald added a subscriber: imp. · View Herald TranscriptOct 16 2020, 4:04 PM
trasz requested review of this revision.Oct 16 2020, 4:04 PM
Harbormaster completed remote builds in B34213: Diff 78320.Oct 16 2020, 4:04 PM
trasz added a reviewer: Linux Emulation.Oct 16 2020, 4:32 PM
trasz added a comment.Oct 19 2020, 11:26 AM

(Tinderboxed).

trasz added a reviewer: kib.Oct 19 2020, 8:54 PM
kib added inline comments.Oct 19 2020, 10:17 PM
sys/compat/linprocfs/linprocfs.c
7

Why ?

trasz added inline comments.Oct 19 2020, 10:33 PM
sys/compat/linprocfs/linprocfs.c
7

It's there in sys/fs/procfs/procfs_mem.c, where I pasted the function from.

kib added inline comments.Oct 20 2020, 1:25 PM
sys/compat/linprocfs/linprocfs.c
7

I do not think it is worth the hassle. Also, I do not see why don't you wrapped procfs_doprocmem.

trasz updated this revision to Diff 78504.Oct 20 2020, 2:11 PM

Reuse procfs_doprocmem() instead of copy/pasting it.

Harbormaster completed remote builds in B34282: Diff 78504.Oct 20 2020, 2:11 PM
trasz marked an inline comment as done.Oct 20 2020, 2:11 PM
kib accepted this revision.Oct 20 2020, 2:20 PM
kib added inline comments.
sys/compat/linprocfs/linprocfs.c
1290

None of these blank lines is warranted or supported by style(9).

This revision is now accepted and ready to land.Oct 20 2020, 2:20 PM
Closed by commit rS366900: Fix linprocfs(4) /proc/self/mem semantics to more closely match Linux. (authored by trasz). · Explain WhyOct 20 2020, 5:24 PM
This revision was automatically updated to reflect the committed changes.
trasz added a commit: rS366900: Fix linprocfs(4) /proc/self/mem semantics to more closely match Linux..
trasz added inline comments.Oct 20 2020, 5:24 PM
sys/compat/linprocfs/linprocfs.c
1290

I find it easier to read this way, though.

kib added inline comments.Oct 20 2020, 11:27 PM
sys/compat/linprocfs/linprocfs.c
1290

The only function of these blank lines is to eat vertical space in editor, allowing to see less code.

Revision Contents
Changeset List

PathSize
sys/
compat/
linprocfs/
36 lines

Diff 78320

View Options

sys/compat/linprocfs/linprocfs.c

Loading...