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
Unknown Object (File)
Feb 29 2024, 10:57 PM
Unknown Object (File)
Feb 19 2024, 9:53 AM
Unknown Object (File)
Jan 13 2024, 11:54 AM
Unknown Object (File)
Jan 10 2024, 1:11 PM
Unknown Object (File)
Dec 20 2023, 7:48 AM
Unknown Object (File)
Dec 17 2023, 2:14 PM
Unknown Object (File)
Dec 13 2023, 4:17 PM
Unknown Object (File)
Nov 19 2023, 3:29 PM
View All 32 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...