BUF/BIO cleanup 4/99:
authorHiten Pandya <hmp@dragonflybsd.org>
Thu, 4 Aug 2005 21:08:35 +0000 (21:08 +0000)
committerHiten Pandya <hmp@dragonflybsd.org>
Thu, 4 Aug 2005 21:08:35 +0000 (21:08 +0000)
Garbage-collect physio_proc_{enter,leave,init} because these functions
only existed for PC-98 compatibility reasons.  This commit is a NOP
and shouldn't make any operational difference.

The purpose of those functions was totally redundant, it just kept a
copy of buf->b_proc and other fields in a separately maintained TAILQ
list called physio_proc_head and physio_proc_feet corresponding to
physio_proc_enter/leave respectively.

The stub definitions and the machine/physio_proc.h header file will
disappear after the drivers using these functions are cleaned up.

sys/i386/include/physio_proc.h
sys/kern/kern_physio.c
sys/platform/pc32/include/physio_proc.h

index fe7bd6b..abf2abb 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/i386/include/physio_proc.h,v 1.1.2.1 2000/10/29 11:05:48 non Exp $    */
-/*     $DragonFly: src/sys/i386/include/Attic/physio_proc.h,v 1.10 2005/06/03 17:14:50 dillon Exp $    */
+/*     $DragonFly: src/sys/i386/include/Attic/physio_proc.h,v 1.11 2005/08/04 21:08:35 hmp Exp $       */
 /*     $NecBSD: physio_proc.h,v 3.4 1999/07/23 20:47:03 honda Exp $    */
 /*     $NetBSD$        */
 
 #include <sys/queue.h>
 #include <sys/thread2.h>
 
-struct physio_proc;
-TAILQ_HEAD(physio_proc_head, physio_proc);
-extern struct physio_proc_head physio_proc_freet, physio_proc_busyt;
-
-struct physio_proc {
-       TAILQ_ENTRY(physio_proc) pp_chain;
-       struct proc *pp_proc;
-};
-
-static __inline struct physio_proc *physio_proc_enter (struct buf *);
-static __inline void physio_proc_leave (struct physio_proc *);
+struct physio_proc { };
 
 static __inline struct physio_proc *
 physio_proc_enter(bp)
        struct buf *bp;
 {
-       struct physio_proc *pp;
-
-       if (bp == NULL || (bp->b_flags & B_PHYS) == 0)
-               return NULL;    
-       if ((pp = TAILQ_FIRST(&physio_proc_freet)) == NULL)
-               return NULL;
-
-       crit_enter();
-       TAILQ_REMOVE(&physio_proc_freet, pp, pp_chain);
-#if !(defined(__DragonFly__) || defined(__FreeBSD__)) || \
-    (defined(__FreeBSD_version) && __FreeBSD_version < 400001)
-       pp->pp_proc = bp->b_proc;
-#endif
-       TAILQ_INSERT_TAIL(&physio_proc_busyt, pp, pp_chain);
-       crit_exit();
-       return pp;
+       return NULL;
 }
 
 static __inline void
 physio_proc_leave(pp)
        struct physio_proc *pp;
 {
-       if (pp == NULL)
-               return;
-
-       crit_enter();
-       TAILQ_REMOVE(&physio_proc_busyt, pp, pp_chain);
-       TAILQ_INSERT_TAIL(&physio_proc_freet, pp, pp_chain);
-       pp->pp_proc = NULL;
-       crit_exit();
+       return;
 }
 
-void physio_proc_init (void);
+static __inline void
+physio_proc_init(void)
+{
+       return;
+}
 #endif /* _MACHINE_PHYSIO_PROC_H_ */
index eb0c3cb..fd608be 100644 (file)
@@ -17,7 +17,7 @@
  *    are met.
  *
  * $FreeBSD: src/sys/kern/kern_physio.c,v 1.46.2.4 2003/11/14 09:51:47 simokawa Exp $
- * $DragonFly: src/sys/kern/kern_physio.c,v 1.11 2005/08/03 16:36:33 hmp Exp $
+ * $DragonFly: src/sys/kern/kern_physio.c,v 1.12 2005/08/04 21:08:35 hmp Exp $
  */
 
 #include <sys/param.h>
 #include <sys/proc.h>
 #include <sys/uio.h>
 #include <sys/device.h>
+#include <sys/thread2.h>
 
 #include <vm/vm.h>
 #include <vm/vm_extern.h>
 
-#include <machine/physio_proc.h>
-
-/* I have no idea what this is used for */
-struct physio_proc_head physio_proc_freet, physio_proc_busyt;
-
 static void
 physwakeup(struct buf *bp)
 {
index 8c194bd..f0521b0 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/i386/include/physio_proc.h,v 1.1.2.1 2000/10/29 11:05:48 non Exp $    */
-/*     $DragonFly: src/sys/platform/pc32/include/Attic/physio_proc.h,v 1.10 2005/06/03 17:14:50 dillon Exp $   */
+/*     $DragonFly: src/sys/platform/pc32/include/Attic/physio_proc.h,v 1.11 2005/08/04 21:08:35 hmp Exp $      */
 /*     $NecBSD: physio_proc.h,v 3.4 1999/07/23 20:47:03 honda Exp $    */
 /*     $NetBSD$        */
 
 #include <sys/queue.h>
 #include <sys/thread2.h>
 
-struct physio_proc;
-TAILQ_HEAD(physio_proc_head, physio_proc);
-extern struct physio_proc_head physio_proc_freet, physio_proc_busyt;
-
-struct physio_proc {
-       TAILQ_ENTRY(physio_proc) pp_chain;
-       struct proc *pp_proc;
-};
-
-static __inline struct physio_proc *physio_proc_enter (struct buf *);
-static __inline void physio_proc_leave (struct physio_proc *);
+struct physio_proc { };
 
 static __inline struct physio_proc *
 physio_proc_enter(bp)
        struct buf *bp;
 {
-       struct physio_proc *pp;
-
-       if (bp == NULL || (bp->b_flags & B_PHYS) == 0)
-               return NULL;    
-       if ((pp = TAILQ_FIRST(&physio_proc_freet)) == NULL)
-               return NULL;
-
-       crit_enter();
-       TAILQ_REMOVE(&physio_proc_freet, pp, pp_chain);
-#if !(defined(__DragonFly__) || defined(__FreeBSD__)) || \
-    (defined(__FreeBSD_version) && __FreeBSD_version < 400001)
-       pp->pp_proc = bp->b_proc;
-#endif
-       TAILQ_INSERT_TAIL(&physio_proc_busyt, pp, pp_chain);
-       crit_exit();
-       return pp;
+       return NULL;
 }
 
 static __inline void
 physio_proc_leave(pp)
        struct physio_proc *pp;
 {
-       if (pp == NULL)
-               return;
-
-       crit_enter();
-       TAILQ_REMOVE(&physio_proc_busyt, pp, pp_chain);
-       TAILQ_INSERT_TAIL(&physio_proc_freet, pp, pp_chain);
-       pp->pp_proc = NULL;
-       crit_exit();
+       return;
 }
 
-void physio_proc_init (void);
+static __inline void
+physio_proc_init(void)
+{
+       return;
+}
 #endif /* _MACHINE_PHYSIO_PROC_H_ */