From nobody Sat Apr 16 02:18:40 2022 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id AA0D05D7A18; Sat, 16 Apr 2022 02:18:41 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4KgH1X6s1zz4m53; Sat, 16 Apr 2022 02:18:40 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1650075521; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=M0BN023002fpbFIcJbgvNbNoOVHEK2i+hzfkzp33FIM=; b=JAWbjOqNhCsbHagzJnDXDYLY8wSiejMK/5q+ssvsdQphJ2AShD8VBkAarCXjSscMYgtFv3 pBCrgdmJbYjW2sWRY+abkCwlxVNDmJY0s64GCT5uNlO1jEYyKFdDfCWx44Xb1HjxVDJ+uE 7Fw0meJz9IcEbfkybzCalUgWx/Q3B38qGeCbNFUVcY7B1KlRxLtjtXrMscZo9wIE6seG7o H9rSiL2PklqBiRQuAHg+Arlxzeda+5Fwc744ZgdNr8CyXhRL35fFq+2CmS7vUm64hLoBX5 h3pA1LBWakCLyPZP6P/BjF5Sgee+iw/CNaIYirDfa9v885uK5MHCRbJIadNqSQ== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id B784D21027; Sat, 16 Apr 2022 02:18:40 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 23G2IedC025349; Sat, 16 Apr 2022 02:18:40 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 23G2IeXu025348; Sat, 16 Apr 2022 02:18:40 GMT (envelope-from git) Date: Sat, 16 Apr 2022 02:18:40 GMT Message-Id: <202204160218.23G2IeXu025348@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Konstantin Belousov Subject: git: 3d0801e86e7e - stable/13 - lockf: remove lf_inode from struct lockf_entry List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: kib X-Git-Repository: src X-Git-Refname: refs/heads/stable/13 X-Git-Reftype: branch X-Git-Commit: 3d0801e86e7ee0207a3011717105b399da78f73e Auto-Submitted: auto-generated ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1650075521; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=M0BN023002fpbFIcJbgvNbNoOVHEK2i+hzfkzp33FIM=; b=KjVPitioK8raMskKeDiW432uTtZkeks5WxB1Dw/4mivRj8B/zdnYLXspOk27Ji1Y8U4zGB 8HbzV5ISDQLvT22U8K+FRdkA5V6A784WEguNoGMuItjhpqOU5fPJxUO+dD4jWnQJzxPpss WmUis76O4GxU+9IoPaAkKgpun4EvHUwrtnqH9L7SAXb1U490AZCsA7JIGaGttOgXjgImho j5P3ys8wHuQ7vWiTMFe+vGYhfhi92TZgVxF0HPRWx/Mfx4r2ROu806ECybW27gGfxjOOfm P3cciHPCirFMDipxRY0LBmbhQNyifGefYXzyWfW9gjRPiO6skmysmuKR/JgJfg== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1650075521; a=rsa-sha256; cv=none; b=n8/6y43o1Ksnb6MRfuKXsR2C+fDXru9ry1uQei+RLxRf1sI6RBdkc5MjmYy2ulb6tkzugU z4J8vypE+XXxaQT6jlMz6yGeLLeXRvBAozaIfxQJ1H2LIQPPzsDRS6ejSfwLqLjB4aXbvQ u3zi+KzVFZfj5li4IYipXjqW7fNRIvQsAT9z+S0BjfvkxDz4hO//MKupHRNNhHsF//+pmN rTQ9RYIGH4RLGK+QQqEGpm8Ka7sWknxolyQNPH+NeISswRaUNXy6hqo/0gSgImFSCA1hne 8xNfcCe1tKxKZ0CSpGkZbERl6ZHgIdtg9NtoN7taVOrc2BwzhLhvZEiqc1Od+Q== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N The branch stable/13 has been updated by kib: URL: https://cgit.FreeBSD.org/src/commit/?id=3d0801e86e7ee0207a3011717105b399da78f73e commit 3d0801e86e7ee0207a3011717105b399da78f73e Author: Konstantin Belousov AuthorDate: 2022-04-01 20:34:59 +0000 Commit: Konstantin Belousov CommitDate: 2022-04-16 02:18:18 +0000 lockf: remove lf_inode from struct lockf_entry (cherry picked from commit 59e85819be69d354fa6f390b791fd9cd06a80f35) --- sys/kern/kern_lockf.c | 20 +++----------------- sys/sys/lockf.h | 1 - 2 files changed, 3 insertions(+), 18 deletions(-) diff --git a/sys/kern/kern_lockf.c b/sys/kern/kern_lockf.c index 8a54e4700d20..079f7b999835 100644 --- a/sys/kern/kern_lockf.c +++ b/sys/kern/kern_lockf.c @@ -571,13 +571,6 @@ retry_setlock: vref(vp); } - /* - * XXX The problem is that VTOI is ufs specific, so it will - * break LOCKF_DEBUG for all other FS's other than UFS because - * it casts the vnode->data ptr to struct inode *. - */ -/* lock->lf_inode = VTOI(ap->a_vp); */ - lock->lf_inode = (struct inode *)0; lock->lf_type = fl->l_type; LIST_INIT(&lock->lf_outedges); LIST_INIT(&lock->lf_inedges); @@ -2498,10 +2491,8 @@ lf_print(char *tag, struct lockf_entry *lock) printf("%s: lock %p for ", tag, (void *)lock); lf_print_owner(lock->lf_owner); - if (lock->lf_inode != (struct inode *)0) - printf(" in ino %ju on dev <%s>,", - (uintmax_t)lock->lf_inode->i_number, - devtoname(ITODEV(lock->lf_inode))); + printf("\nvnode %p", lock->lf_vnode); + VOP_PRINT(lock->lf_vnode); printf(" %s, start %jd, end ", lock->lf_type == F_RDLCK ? "shared" : lock->lf_type == F_WRLCK ? "exclusive" : @@ -2524,12 +2515,7 @@ lf_printlist(char *tag, struct lockf_entry *lock) struct lockf_entry *lf, *blk; struct lockf_edge *e; - if (lock->lf_inode == (struct inode *)0) - return; - - printf("%s: Lock list for ino %ju on dev <%s>:\n", - tag, (uintmax_t)lock->lf_inode->i_number, - devtoname(ITODEV(lock->lf_inode))); + printf("%s: Lock list for vnode %p:\n", tag, lock->lf_vnode); LIST_FOREACH(lf, &lock->lf_vnode->v_lockf->ls_active, lf_link) { printf("\tlock %p for ",(void *)lf); lf_print_owner(lock->lf_owner); diff --git a/sys/sys/lockf.h b/sys/sys/lockf.h index 368c773283ec..cca02501a462 100644 --- a/sys/sys/lockf.h +++ b/sys/sys/lockf.h @@ -77,7 +77,6 @@ struct lockf_entry { off_t lf_end; /* (s) Byte # of the end of the lock (OFF_MAX=EOF) */ struct lock_owner *lf_owner; /* (c) Owner of the lock */ struct vnode *lf_vnode; /* (c) File being locked (only valid for active lock) */ - struct inode *lf_inode; /* (c) Back pointer to the inode */ struct task *lf_async_task;/* (c) Async lock callback */ LIST_ENTRY(lockf_entry) lf_link; /* (s) Linkage for lock lists */ struct lockf_edge_list lf_outedges; /* (s) list of out-edges */