Page MenuHomeFreeBSD

shm_alloc(): cleanup
ClosedPublic

Authored by kib on Oct 8 2024, 5:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 6, 2:27 PM
Unknown Object (File)
Feb 16 2026, 5:31 AM
Unknown Object (File)
Feb 8 2026, 4:37 AM
Unknown Object (File)
Feb 7 2026, 6:25 PM
Unknown Object (File)
Feb 7 2026, 4:30 PM
Unknown Object (File)
Jan 31 2026, 1:26 PM
Unknown Object (File)
Dec 27 2025, 10:46 AM
Unknown Object (File)
Nov 28 2025, 10:24 PM
Subscribers

Details

Summary
Consistently use the shorter name 'obj' for the new object.
Set OBJ_POSIXSHM flag outside if, it is set on both pathes.

Suggested by:   alc

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib requested review of this revision.Oct 8 2024, 5:50 PM

I think that unlocked assignment to swp/phys_priv is fine,the field is never changed after the initialization. I would even set the flag lockless, since the object is not yet visible, but I think the consistency is useful there, e.g. for sanitizers.

sys/kern/uipc_shm.c
947–948

The assignment to shmfd->shm_object here isn't needed.

kib marked an inline comment as done.

Remove unneeded assignment.

This revision is now accepted and ready to land.Oct 9 2024, 8:10 AM
This revision was automatically updated to reflect the committed changes.