Page MenuHomeFreeBSD

tarfs: Correctly track link count.
ClosedPublic

Authored by des on Mar 10 2023, 5:50 PM.
Tags
None
Referenced Files
F108748196: D39019.id118908.diff
Mon, Jan 27, 5:00 PM
Unknown Object (File)
Thu, Jan 23, 6:41 PM
Unknown Object (File)
Thu, Jan 23, 6:33 PM
Unknown Object (File)
Thu, Jan 16, 5:17 AM
Unknown Object (File)
Fri, Jan 10, 5:00 PM
Unknown Object (File)
Thu, Jan 9, 9:15 AM
Unknown Object (File)
Thu, Jan 9, 9:14 AM
Unknown Object (File)
Wed, Jan 8, 6:55 AM
Subscribers

Details

Summary

Sponsored by: Juniper Networks, Inc.
Sponsored by: Klara, Inc.

Diff Detail

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

Event Timeline

des requested review of this revision.Mar 10 2023, 5:50 PM

buggy, will resubmit

sys/fs/tarfs/tarfs_vfsops.c
745

What protects other->nlink during the increment? Could two calls to tarfs_alloc_one() executed in parallel, targeting the same dst node?

Or even, what protects other itself? For instance, could unmount occur in parallel with the tarfs_alloc_one()?

sys/fs/tarfs/tarfs_vfsops.c
745

tarfs_alloc_one() is called sequentially during mount, and tarfs_free_node() is called sequentially during unmount or while cleaning up after a failed mount.

This revision is now accepted and ready to land.Mar 13 2023, 5:56 PM
This revision was automatically updated to reflect the committed changes.