git: ec70e1177b20 - stable/14 - p9fs: remove duplicated code
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Fri, 04 Jul 2025 13:28:50 UTC
The branch stable/14 has been updated by arrowd:
URL: https://cgit.FreeBSD.org/src/commit/?id=ec70e1177b2004a5990222dbe3bd6130d9566822
commit ec70e1177b2004a5990222dbe3bd6130d9566822
Author: Danilo Egea Gondolfo <danilo@FreeBSD.org>
AuthorDate: 2024-07-09 19:07:18 +0000
Commit: Gleb Popov <arrowd@FreeBSD.org>
CommitDate: 2025-07-04 13:28:06 +0000
p9fs: remove duplicated code
This code is using the vnode after it has been released and causing a
panic when a p9fs shared volume is unmounted. In fact, it seems like it's
just duplicated code left behind from a bad merge.
PR: 279887
Reported by: Michael Dexter
Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/1323
(cherry picked from commit a6ca6dfd60b66eec563bd473d96b31f0be1de80a)
---
sys/fs/p9fs/p9fs_vnops.c | 10 ----------
1 file changed, 10 deletions(-)
diff --git a/sys/fs/p9fs/p9fs_vnops.c b/sys/fs/p9fs/p9fs_vnops.c
index 5556e60f4bae..6ac0a7c008cc 100644
--- a/sys/fs/p9fs/p9fs_vnops.c
+++ b/sys/fs/p9fs/p9fs_vnops.c
@@ -125,16 +125,6 @@ p9fs_cleanup(struct p9fs_node *np)
/* Destroy the vm object and flush associated pages. */
vnode_destroy_vobject(vp);
- /* Remove the vnode from hash list if vnode is not already deleted */
- if ((np->flags & P9FS_NODE_DELETED) == 0)
- vfs_hash_remove(vp);
-
- /* Invalidate all entries to a particular vnode. */
- cache_purge(vp);
-
- /* Destroy the vm object and flush associated pages. */
- vnode_destroy_vobject(vp);
-
/* Remove all the FID */
p9fs_fid_remove_all(np, FALSE);