From owner-freebsd-current@FreeBSD.ORG Wed Oct 12 14:21:02 2005 Return-Path: X-Original-To: freebsd-current@freebsd.org Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 2F94C16A420; Wed, 12 Oct 2005 14:21:02 +0000 (GMT) (envelope-from rwatson@FreeBSD.org) Received: from cyrus.watson.org (cyrus.watson.org [209.31.154.42]) by mx1.FreeBSD.org (Postfix) with ESMTP id B431643D73; Wed, 12 Oct 2005 14:21:01 +0000 (GMT) (envelope-from rwatson@FreeBSD.org) Received: from fledge.watson.org (fledge.watson.org [209.31.154.41]) by cyrus.watson.org (Postfix) with ESMTP id BAAAD46BB1; Wed, 12 Oct 2005 10:21:00 -0400 (EDT) Date: Wed, 12 Oct 2005 15:21:00 +0100 (BST) From: Robert Watson X-X-Sender: robert@fledge.watson.org To: Michael Nottebrock In-Reply-To: <200510120041.03079.lofi@freebsd.org> Message-ID: <20051012150600.O48006@fledge.watson.org> References: <200509220113.06667.lofi@freebsd.org> <20050929221041.H34322@fledge.watson.org> <200509292335.52277.lofi@freebsd.org> <200510120041.03079.lofi@freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: jeff@FreeBSD.org, freebsd-current@freebsd.org, csjp@FreeBSD.org Subject: Re: UFS_EXTATTR(_AUTOSTART) broken on 6.0-BETA5 X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 12 Oct 2005 14:21:02 -0000 On Wed, 12 Oct 2005, Michael Nottebrock wrote: > On Thursday, 29. September 2005 23:35, Michael Nottebrock wrote: > >>> I may have the opportunity to work on this this weekend, but not >>> entirely clear. If Christian is already running with it, I'd much >>> rather let him take this one on as I have a lot of other things on my >>> plate. >> >> Just wanted to make sure it didn't get lost in mailing list traffic, >> thanks! > > Still panics in 6.0RC1. Could you try the attached patch? It appears to work for me here, and I've committed it as ufs_extattr.c:1.82 in HEAD due to a high confidence it's the right fix. I will MFC it for 6.x if it also works for you. Given that extended attributes seem to be getting little testing with UFS1 at this point, I'd appreciate it if you could exercise them as much as possible in testing so we can try to catch any other bugs that may have crept in as the infrastructure around this code changed. Thanks, Robert N M Watson Index: ufs_extattr.c =================================================================== RCS file: /home/ncvs/src/sys/ufs/ufs/ufs_extattr.c,v retrieving revision 1.81 retrieving revision 1.82 diff -u -r1.81 -r1.82 --- ufs_extattr.c 31 Mar 2005 05:58:14 -0000 1.81 +++ ufs_extattr.c 12 Oct 2005 14:18:58 -0000 1.82 @@ -251,6 +251,7 @@ cnp.cn_flags = ISLASTCN; if (lockparent == UE_GETDIR_LOCKPARENT) cnp.cn_flags |= LOCKPARENT; + cnp.cn_lkflags = LK_EXCLUSIVE; cnp.cn_thread = td; cnp.cn_cred = td->td_ucred; cnp.cn_pnbuf = uma_zalloc(namei_zone, M_WAITOK);