bring cloning up too date; Munge it so it will work with atleast

oga@'s upcoming DRM changes and too some degree ratchov@'s audio
work. It still works for bpf's though.

Parts from ratchov@; fstat(1) parts from Pedro Martelletto;
tested by many, ok'ed by a few;
"get going with cloning" deraadt@

3 files changed, 61 insertions, 17 deletions

diff --git a/sys/miscfs/specfs/spec_subr.c b/sys/miscfs/specfs/spec_subr.c
index 399aa02c7b1..2077afe0b35 100644
--- a/sys/miscfs/specfs/spec_subr.c
+++ b/sys/miscfs/specfs/spec_subr.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: spec_subr.c,v 1.3 2007/12/09 20:54:01 jmc Exp $	*/
+/*	$OpenBSD: spec_subr.c,v 1.4 2008/04/08 14:46:45 thib Exp $	*/
 
 /*
  * Copyright (c) 2006 Pedro Martelletto <pedro@ambientworks.net>
@@ -25,6 +25,12 @@
 
 #include <miscfs/specfs/specdev.h>
 
+#ifdef	CLONE_DEBUG
+#define	DNPRINTF(m...)	do { printf(m);  } while (0)
+#else
+#define	DNPRINTF(m...)	/* nothing */
+#endif
+
 int
 spec_open_clone(struct vop_open_args *ap)
 {
@@ -32,6 +38,8 @@ spec_open_clone(struct vop_open_args *ap)
 	struct cloneinfo *cip;
 	int error, i;
 
+	DNPRINTF("cloning vnode\n");
+
 	for (i = 1; i < sizeof(vp->v_specbitmap) * NBBY; i++)
 		if (isclr(vp->v_specbitmap, i)) {
 			setbit(vp->v_specbitmap, i);
@@ -41,9 +49,6 @@ spec_open_clone(struct vop_open_args *ap)
 	if (i == sizeof(vp->v_specbitmap) * NBBY)
 		return (EBUSY); /* too many open instances */
 
-	printf("spec_open_clone(): cloning device (%d, %d) for pid %u\n",
-	    major(vp->v_rdev), minor(vp->v_rdev), curproc->p_pid);
-
 	error = cdevvp(makedev(major(vp->v_rdev), i), &cvp);
 	if (error)
 		return (error); /* out of vnodes */
@@ -60,6 +65,8 @@ spec_open_clone(struct vop_open_args *ap)
 		 return (error); /* device open failed */
 	}
 
+	cvp->v_flag |= VCLONE;
+
 	cip = malloc(sizeof(struct cloneinfo), M_TEMP, M_WAITOK);
 	cip->ci_data = vp->v_data;
 	cip->ci_vp = cvp;
@@ -68,8 +75,7 @@ spec_open_clone(struct vop_open_args *ap)
 	vp->v_flag |= VCLONED;
 	vp->v_data = cip;
 
-	printf("spec_open_clone(): new minor for cloned device is %d\n",
-	    minor(cvp->v_rdev));
+	DNPRINTF("clone of vnode %p is vnode %p\n", vp, cvp);
 
 	return (0); /* device cloned */
 }
@@ -87,9 +93,7 @@ spec_close_clone(struct vop_close_args *ap)
 
 	pvp = vp->v_specparent; /* get parent device */
 	clrbit(pvp->v_specbitmap, minor(vp->v_rdev));
-
-	printf("spec_close_clone(): freeing minor %d of dev %d for"
-	    " pid %u\n", minor(vp->v_rdev), major(vp->v_rdev), curproc->p_pid);
+	vrele(pvp);
 
 	return (0); /* clone closed */
 }
diff --git a/sys/miscfs/specfs/spec_vnops.c b/sys/miscfs/specfs/spec_vnops.c
index 97bfdf61f85..e5d3f2c86f0 100644
--- a/sys/miscfs/specfs/spec_vnops.c
+++ b/sys/miscfs/specfs/spec_vnops.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: spec_vnops.c,v 1.44 2007/12/27 13:59:12 thib Exp $	*/
+/*	$OpenBSD: spec_vnops.c,v 1.45 2008/04/08 14:46:45 thib Exp $	*/
 /*	$NetBSD: spec_vnops.c,v 1.29 1996/04/22 01:42:38 christos Exp $	*/
 
 /*
@@ -529,6 +529,8 @@ spec_close(void *v)
 			vrele(vp);
 			ap->a_p->p_session->s_ttyvp = NULL;
 		}
+		if (cdevsw[major(dev)].d_flags & D_CLONE)
+			return (spec_close_clone(ap));
 		/*
 		 * If the vnode is locked, then we are in the midst
 		 * of forcably closing the device, otherwise we only
@@ -536,8 +538,6 @@ spec_close(void *v)
 		 */
 		if (vcount(vp) > 1 && (vp->v_flag & VXLOCK) == 0)
 			return (0);
-		if (cdevsw[major(dev)].d_flags & D_CLONE)
-			return (spec_close_clone(ap));
 		devclose = cdevsw[major(dev)].d_close;
 		mode = S_IFCHR;
 		break;
@@ -579,6 +579,47 @@ spec_close(void *v)
 	return ((*devclose)(dev, ap->a_fflag, mode, ap->a_p));
 }
 
+int
+spec_getattr(void *v)
+{
+	struct vop_getattr_args	*ap = v;
+	struct vnode		*vp = ap->a_vp;
+
+	if (!(vp->v_flag & VCLONE))
+		return (EBADF);
+
+	return (VOP_GETATTR(vp->v_specparent, ap->a_vap, ap->a_cred, ap->a_p));
+}
+
+int
+spec_setattr(void *v)
+{
+	struct vop_getattr_args	*ap = v;
+	struct vnode		*vp = ap->a_vp;
+	int			 error;
+
+	if (!(vp->v_flag & VCLONE))
+		return (EBADF);
+
+	vn_lock(vp->v_specparent, LK_EXCLUSIVE|LK_RETRY, ap->a_p);
+	error = VOP_SETATTR(vp->v_specparent, ap->a_vap, ap->a_cred, ap->a_p);
+	VOP_UNLOCK(vp, 0, ap->a_p);
+
+	return (error);
+}
+
+int
+spec_access(void *v)
+{
+	struct vop_access_args	*ap = v;
+	struct vnode		*vp = ap->a_vp;
+
+	if (!(vp->v_flag & VCLONE))
+		return (EBADF);
+
+	return (VOP_ACCESS(vp->v_specparent, ap->a_mode, ap->a_cred, ap->a_p));
+}
+
 /*
  * Print out the contents of a special device vnode.
  */
diff --git a/sys/miscfs/specfs/specdev.h b/sys/miscfs/specfs/specdev.h
index fc40a7e2805..98eb73cfe8d 100644
--- a/sys/miscfs/specfs/specdev.h
+++ b/sys/miscfs/specfs/specdev.h
@@ -1,4 +1,4 @@
-/*	$OpenBSD: specdev.h,v 1.21 2007/12/27 13:59:12 thib Exp $	*/
+/*	$OpenBSD: specdev.h,v 1.22 2008/04/08 14:46:45 thib Exp $	*/
 /*	$NetBSD: specdev.h,v 1.12 1996/02/13 13:13:01 mycroft Exp $	*/
 
 /*
@@ -91,14 +91,13 @@ struct	uio;
 
 int	spec_badop(void *);
 int	spec_ebadf(void *);
-
+int	spec_getattr(void *);
+int	spec_setattr(void *);
+int	spec_access(void *);
 #define	spec_create	spec_badop
 #define	spec_mknod	spec_badop
 int	spec_open(void *);
 int	spec_close(void *);
-#define	spec_access	spec_ebadf
-#define	spec_getattr	spec_ebadf
-#define	spec_setattr	spec_ebadf
 int	spec_read(void *);
 int	spec_write(void *);
 int	spec_ioctl(void *);