From owner-freebsd-hackers@FreeBSD.ORG  Wed Apr 27 02:44:40 2005
Return-Path: <owner-freebsd-hackers@FreeBSD.ORG>
Delivered-To: freebsd-hackers@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id 15FB316A4CE; Wed, 27 Apr 2005 02:44:40 +0000 (GMT)
Received: from f40.mail.ru (f40.mail.ru [194.67.57.78])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 6F87143D41; Wed, 27 Apr 2005 02:44:39 +0000 (GMT)
	(envelope-from shmukler@mail.ru)
Received: from mail by f40.mail.ru with local 
	id 1DQcXV-0005rT-00; Wed, 27 Apr 2005 06:44:37 +0400
Received: from [24.184.137.0] by win.mail.ru with HTTP;
	Wed, 27 Apr 2005 06:44:37 +0400
From: Igor Shmukler <shmukler@mail.ru>
To: hackers@freebsd.org
Mime-Version: 1.0
X-Mailer: mPOP Web-Mail 2.19
X-Originating-IP: [24.184.137.0]
Date: Wed, 27 Apr 2005 06:44:37 +0400
Content-Type: text/plain; charset=koi8-r
Content-Transfer-Encoding: 8bit
Message-Id: <E1DQcXV-0005rT-00.shmukler-mail-ru@f40.mail.ru>
cc: rwatson@freebsd.org
Subject: vn_fullpath() and devices
X-BeenThere: freebsd-hackers@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
Reply-To: Igor Shmukler <shmukler@mail.ru>
List-Id: Technical Discussions relating to FreeBSD
	<freebsd-hackers.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-hackers>,
	<mailto:freebsd-hackers-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-hackers>
List-Post: <mailto:freebsd-hackers@freebsd.org>
List-Help: <mailto:freebsd-hackers-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-hackers>,
	<mailto:freebsd-hackers-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 27 Apr 2005 02:44:40 -0000

hello,

i reported before that vn_fullpath() does not currently deal with VCHR type of vnodes.

There is an easy solution for this:

                 if (vnp->v_type == VCHR) {
                           fullpath = vnp->v_rdev->si_name;
                           VOP_UNLOCK(vnp, 0, td);
                           len = sizeof("/dev/") + strlen(fullpath);
  	                        freepath = vdt_malloc(len);
  	                        sprintf(freepath, "/dev/%s", fullpath);
  	                        fullpath = freepath;
                 } else {

it this works for everyone, i could make and test a patch against whatever branch is 
appropriate.

thank you,
igor