Page MenuHomeFreeBSD

shm_alloc(): cleanup
ClosedPublic

Authored by kib on Oct 8 2024, 5:50 PM.
Tags
None
Referenced Files
F163016688: D47018.diff
Sun, Jul 19, 5:30 AM
Unknown Object (File)
Thu, Jul 9, 12:40 AM
Unknown Object (File)
Thu, Jun 25, 11:24 AM
Unknown Object (File)
Jun 18 2026, 10:41 PM
Unknown Object (File)
May 22 2026, 2:26 PM
Unknown Object (File)
May 9 2026, 5:46 PM
Unknown Object (File)
May 9 2026, 5:46 PM
Unknown Object (File)
May 6 2026, 10:27 AM
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.