Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 7 Sep 2012 18:41:19 +0000 (UTC)
From:      John Baldwin <jhb@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r240209 - head/sys/dev/twe
Message-ID:  <201209071841.q87IfJEF045693@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: jhb
Date: Fri Sep  7 18:41:19 2012
New Revision: 240209
URL: http://svn.freebsd.org/changeset/base/240209

Log:
  Remove remaining 4.x compat shims.  No resulting changes (verified by
  md5).

Modified:
  head/sys/dev/twe/twe.c
  head/sys/dev/twe/twe_compat.h
  head/sys/dev/twe/twe_freebsd.c
  head/sys/dev/twe/twevar.h

Modified: head/sys/dev/twe/twe.c
==============================================================================
--- head/sys/dev/twe/twe.c	Fri Sep  7 18:03:29 2012	(r240208)
+++ head/sys/dev/twe/twe.c	Fri Sep  7 18:41:19 2012	(r240209)
@@ -400,7 +400,7 @@ twe_startio(struct twe_softc *sc)
 {
     struct twe_request	*tr;
     TWE_Command		*cmd;
-    twe_bio		*bp;
+    struct bio		*bp;
     int			error;
 
     debug_called(4);
@@ -431,10 +431,10 @@ twe_startio(struct twe_softc *sc)
 	    /* connect the bio to the command */
 	    tr->tr_complete = twe_completeio;
 	    tr->tr_private = bp;
-	    tr->tr_data = TWE_BIO_DATA(bp);
-	    tr->tr_length = TWE_BIO_LENGTH(bp);
+	    tr->tr_data = bp->bio_data;
+	    tr->tr_length = bp->bio_bcount;
 	    cmd = TWE_FIND_COMMAND(tr);
-	    if (TWE_BIO_IS_READ(bp)) {
+	    if (bp->bio_cmd == BIO_READ) {
 		tr->tr_flags |= TWE_CMD_DATAIN;
 		cmd->io.opcode = TWE_OP_READ;
 	    } else {
@@ -444,9 +444,9 @@ twe_startio(struct twe_softc *sc)
 	
 	    /* build a suitable I/O command (assumes 512-byte rounded transfers) */
 	    cmd->io.size = 3;
-	    cmd->io.unit = TWE_BIO_UNIT(bp);
+	    cmd->io.unit = *(int *)(bp->bio_driver1);
 	    cmd->io.block_count = (tr->tr_length + TWE_BLOCK_SIZE - 1) / TWE_BLOCK_SIZE;
-	    cmd->io.lba = TWE_BIO_LBA(bp);
+	    cmd->io.lba = bp->bio_pblkno;
 	}
 	
 	/* did we find something to do? */
@@ -461,8 +461,9 @@ twe_startio(struct twe_softc *sc)
 		break;
 	    tr->tr_status = TWE_CMD_ERROR;
 	    if (tr->tr_private != NULL) {
-		bp = (twe_bio *)(tr->tr_private);
-		TWE_BIO_SET_ERROR(bp, error);
+		bp = (struct bio *)(tr->tr_private);
+		bp->bio_error = error;
+		bp->bio_flags |= BIO_ERROR;
 		tr->tr_private = NULL;
 		twed_intr(bp);
 	        twe_release_request(tr);
@@ -1012,15 +1013,17 @@ twe_completeio(struct twe_request *tr)
 {
     TWE_Command		*cmd = TWE_FIND_COMMAND(tr);
     struct twe_softc	*sc = tr->tr_sc;
-    twe_bio		*bp = (twe_bio *)tr->tr_private;
+    struct bio		*bp = tr->tr_private;
 
     debug_called(4);
 
     if (tr->tr_status == TWE_CMD_COMPLETE) {
 
 	if (cmd->generic.status)
-	    if (twe_report_request(tr))
-		TWE_BIO_SET_ERROR(bp, EIO);
+	    if (twe_report_request(tr)) {
+		bp->bio_error = EIO;
+		bp->bio_flags |= BIO_ERROR;
+	    }
 
     } else {
 	twe_panic(sc, "twe_completeio on incomplete command");

Modified: head/sys/dev/twe/twe_compat.h
==============================================================================
--- head/sys/dev/twe/twe_compat.h	Fri Sep  7 18:03:29 2012	(r240208)
+++ head/sys/dev/twe/twe_compat.h	Fri Sep  7 18:41:19 2012	(r240209)
@@ -49,6 +49,7 @@
 #include <sys/sysctl.h>
 #include <sys/sx.h>
 
+#include <sys/bio.h>
 #include <sys/bus.h>
 #include <sys/conf.h>
 #include <sys/disk.h>
@@ -61,6 +62,8 @@
 #include <dev/pci/pcireg.h>
 #include <dev/pci/pcivar.h>
 
+#include <geom/geom_disk.h>
+
 #define TWE_DRIVER_NAME		twe
 #define TWED_DRIVER_NAME	twed
 #define TWE_MALLOC_CLASS	M_TWE
@@ -108,27 +111,6 @@
 #define twe_printf(sc, fmt, args...)	device_printf(sc->twe_dev, fmt , ##args)
 #define twed_printf(twed, fmt, args...)	device_printf(twed->twed_dev, fmt , ##args)
 
-# include <sys/bio.h>
-# include <geom/geom_disk.h>
-typedef struct bio			twe_bio;
-typedef struct bio_queue_head		twe_bioq;
-# define TWE_BIO_QINIT(bq)		bioq_init(&bq);
-# define TWE_BIO_QINSERT(bq, bp)	bioq_insert_tail(&bq, bp)
-# define TWE_BIO_QFIRST(bq)		bioq_first(&bq)
-# define TWE_BIO_QREMOVE(bq, bp)	bioq_remove(&bq, bp)
-# define TWE_BIO_IS_READ(bp)		((bp)->bio_cmd == BIO_READ)
-# define TWE_BIO_DATA(bp)		(bp)->bio_data
-# define TWE_BIO_LENGTH(bp)		(bp)->bio_bcount
-# define TWE_BIO_LBA(bp)		(bp)->bio_pblkno
-# define TWE_BIO_SOFTC(bp)		(bp)->bio_disk->d_drv1
-# define TWE_BIO_UNIT(bp)		*(int *)(bp->bio_driver1)
-# define TWE_BIO_SET_ERROR(bp, err)	do { (bp)->bio_error = err; (bp)->bio_flags |= BIO_ERROR;} while(0)
-# define TWE_BIO_HAS_ERROR(bp)		((bp)->bio_flags & BIO_ERROR)
-# define TWE_BIO_RESID(bp)		(bp)->bio_resid
-# define TWE_BIO_DONE(bp)		biodone(bp)
-# define TWE_BIO_STATS_START(bp)
-# define TWE_BIO_STATS_END(bp)
-
 #define	TWE_IO_LOCK(sc)			mtx_lock(&(sc)->twe_io_lock)
 #define	TWE_IO_UNLOCK(sc)		mtx_unlock(&(sc)->twe_io_lock)
 #define	TWE_IO_ASSERT_LOCKED(sc)	mtx_assert(&(sc)->twe_io_lock, MA_OWNED)

Modified: head/sys/dev/twe/twe_freebsd.c
==============================================================================
--- head/sys/dev/twe/twe_freebsd.c	Fri Sep  7 18:03:29 2012	(r240208)
+++ head/sys/dev/twe/twe_freebsd.c	Fri Sep  7 18:41:19 2012	(r240209)
@@ -720,9 +720,9 @@ twed_open(struct disk *dp)
  * Handle an I/O request.
  */
 static void
-twed_strategy(twe_bio *bp)
+twed_strategy(struct bio *bp)
 {
-    struct twed_softc	*sc = (struct twed_softc *)TWE_BIO_SOFTC(bp);
+    struct twed_softc	*sc = bp->bio_disk->d_drv1;
 
     debug_called(4);
 
@@ -731,16 +731,14 @@ twed_strategy(twe_bio *bp)
 
     /* bogus disk? */
     if (sc == NULL || sc->twed_drive->td_disk == NULL) {
-	TWE_BIO_SET_ERROR(bp, EINVAL);
+	bp->bio_error = EINVAL;
+	bp->bio_flags |= BIO_ERROR;
 	printf("twe: bio for invalid disk!\n");
-	TWE_BIO_DONE(bp);
+	biodone(bp);
 	TWED_BIO_OUT;
 	return;
     }
 
-    /* perform accounting */
-    TWE_BIO_STATS_START(bp);
-
     /* queue the bio on the controller */
     TWE_IO_LOCK(sc->twed_controller);
     twe_enqueue_bio(sc->twed_controller, bp);
@@ -779,16 +777,15 @@ twed_dump(void *arg, void *virtual, vm_o
  * Handle completion of an I/O request.
  */
 void
-twed_intr(twe_bio *bp)
+twed_intr(struct bio *bp)
 {
     debug_called(4);
 
     /* if no error, transfer completed */
-    if (!TWE_BIO_HAS_ERROR(bp))
-	TWE_BIO_RESID(bp) = 0;
+    if (!(bp->bio_flags & BIO_ERROR))
+	bp->bio_resid = 0;
 
-    TWE_BIO_STATS_END(bp);
-    TWE_BIO_DONE(bp);
+    biodone(bp);
     TWED_BIO_OUT;
 }
 

Modified: head/sys/dev/twe/twevar.h
==============================================================================
--- head/sys/dev/twe/twevar.h	Fri Sep  7 18:03:29 2012	(r240208)
+++ head/sys/dev/twe/twevar.h	Fri Sep  7 18:41:19 2012	(r240209)
@@ -114,7 +114,7 @@ struct twe_softc 
 {
     /* controller queues and arrays */
     TAILQ_HEAD(, twe_request)	twe_free;			/* command structures available for reuse */
-    twe_bioq 			twe_bioq;			/* outstanding I/O operations */
+    struct bio_queue_head	twe_bioq;			/* outstanding I/O operations */
     TAILQ_HEAD(, twe_request)	twe_ready;			/* requests ready for the controller */
     TAILQ_HEAD(, twe_request)	twe_busy;			/* requests busy in the controller */
     TAILQ_HEAD(, twe_request)	twe_complete;			/* active commands (busy or waiting for completion) */
@@ -166,7 +166,7 @@ extern int	twe_detach_drive(struct twe_s
 					 int unit);		/* detach drive */
 extern void	twe_clear_pci_parity_error(struct twe_softc *sc);
 extern void	twe_clear_pci_abort(struct twe_softc *sc);
-extern void	twed_intr(twe_bio *bp);				/* return bio from core */
+extern void	twed_intr(struct bio *bp);				/* return bio from core */
 extern struct twe_request *twe_allocate_request(struct twe_softc *sc, int tag);	/* allocate request structure */
 extern void	twe_free_request(struct twe_request *tr);	/* free request structure */
 extern int	twe_map_request(struct twe_request *tr);	/* make request visible to controller, do s/g */
@@ -249,24 +249,24 @@ TWEQ_REQUEST_QUEUE(complete, TWEQ_COMPLE
 static __inline void
 twe_initq_bio(struct twe_softc *sc)
 {
-    TWE_BIO_QINIT(sc->twe_bioq);
+    bioq_init(&sc->twe_bioq);
     TWEQ_INIT(sc, TWEQ_BIO);
 }
 
 static __inline void
-twe_enqueue_bio(struct twe_softc *sc, twe_bio *bp)
+twe_enqueue_bio(struct twe_softc *sc, struct bio *bp)
 {
-    TWE_BIO_QINSERT(sc->twe_bioq, bp);
+    bioq_insert_tail(&sc->twe_bioq, bp);
     TWEQ_ADD(sc, TWEQ_BIO);
 }
 
-static __inline twe_bio *
+static __inline struct bio *
 twe_dequeue_bio(struct twe_softc *sc)
 {
-    twe_bio	*bp;
+    struct bio	*bp;
 
-    if ((bp = TWE_BIO_QFIRST(sc->twe_bioq)) != NULL) {
-	TWE_BIO_QREMOVE(sc->twe_bioq, bp);
+    if ((bp = bioq_first(&sc->twe_bioq)) != NULL) {
+	bioq_remove(&sc->twe_bioq, bp);
 	TWEQ_REMOVE(sc, TWEQ_BIO);
     }
     return(bp);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201209071841.q87IfJEF045693>