 @@ -1,14 +1,14 @@
-/* $NetBSD: ld_thunkbus.c,v 1.21 2011/12/13 13:32:15 jmcneill Exp $ */
+/* $NetBSD: ld_thunkbus.c,v 1.22 2011/12/13 15:50:17 reinoud Exp $ */
 /*-
  * Copyright (c) 2011 Jared D. McNeill <jmcneill@invisible.ca>
  * All rights reserved.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
 @@ -17,27 +17,27 @@
  * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
  * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
  * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
  * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
  * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
  * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
  * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
  * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
  * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
  * POSSIBILITY OF SUCH DAMAGE.
  */
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ld_thunkbus.c,v 1.21 2011/12/13 13:32:15 jmcneill Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ld_thunkbus.c,v 1.22 2011/12/13 15:50:17 reinoud Exp $");
 #include <sys/param.h>
 #include <sys/proc.h>
 #include <sys/systm.h>
 #include <sys/device.h>
 #include <sys/buf.h>
 #include <sys/disk.h>
 #include <sys/kmem.h>
 #include <dev/ldvar.h>
 #include <machine/mainbus.h>
 #include <machine/thunk.h>
 @@ -55,28 +55,26 @@ static void	ld_thunkbus_complete(void *a
 struct ld_thunkbus_softc;
 struct ld_thunkbus_transfer {
 	struct ld_thunkbus_softc *tt_sc;
 	struct buf	*tt_bp;
 };
 struct ld_thunkbus_softc {
 	struct ld_softc	sc_ld;
 	int		sc_fd;
 	void		*sc_ih;
 	uint8_t		sc_bbuf[MAXBSIZE];
 	struct ld_thunkbus_transfer sc_tt;
 	bool		busy;
 };
 CFATTACH_DECL_NEW(ld_thunkbus, sizeof(struct ld_thunkbus_softc),
     ld_thunkbus_match, ld_thunkbus_attach, NULL, NULL);
 static int
 ld_thunkbus_match(device_t parent, cfdata_t match, void *opaque)
+{
 	struct thunkbus_attach_args *taa = opaque;
 	if (taa->taa_type != THUNKBUS_TYPE_DISKIMAGE)
 @@ -142,62 +140,46 @@ ld_thunkbus_ldstart(struct ld_softc *ld,
 	error = 0;
 	return error == -1 ? thunk_geterrno() : 0;
+}
 static void
 ld_thunkbus_complete(void *arg)
+{
 	struct ld_softc *ld = arg;
 	struct ld_thunkbus_softc *sc = (struct ld_thunkbus_softc *)ld;
 	struct ld_thunkbus_transfer *tt = &sc->sc_tt;
 	struct buf *bp = tt->tt_bp;
 	off_t offset = bp->b_rawblkno * ld->sc_secsize;
 	void *bbuf = sc->sc_bbuf;
 	size_t ret;
 	if (!sc->busy)
 		panic("%s: but not busy?\n", __func__);
 	if (offset < 0 ||
 	    offset + bp->b_bcount > ld->sc_secsize * ld->sc_secperunit) {
 		bp->b_error = EIO;
 		bp->b_resid = bp->b_bcount;
 		goto done;
+	}
 	//printf("ld: %s %u @ %lld -> %p (flags 0x%08x)\n", bp->b_flags & B_READ ? "read" : "write",
 	//    (unsigned int)bp->b_bcount, (long long)offset, bp->b_data, bp->b_flags);
 	/* read/write the request */
 	if (bp->b_flags & B_READ) {
-		if (bp->b_flags & B_PHYS) {
+		ret = thunk_pread(sc->sc_fd, bp->b_data, bp->b_bcount, offset);
 			/* read to bounce buffer and copy out */
 			ret = thunk_pread(sc->sc_fd, bbuf, bp->b_bcount, offset);
 			if (ret > 0)
 				copyout(bbuf, bp->b_data, ret);
 		} else {
 			/* just read it */
 			ret = thunk_pread(sc->sc_fd, bp->b_data, bp->b_bcount, offset);
 	} else {
-		if (bp->b_flags & B_PHYS) {
+		ret = thunk_pwrite(sc->sc_fd, bp->b_data, bp->b_bcount, offset);
 			/* copy in to bounce buffer and write it */
 			copyin(bp->b_data, bbuf, bp->b_bcount);
 			ret = thunk_pwrite(sc->sc_fd, bbuf, bp->b_bcount, offset);
 		} else {
 			/* just write it */
 			ret = thunk_pwrite(sc->sc_fd, bp->b_data, bp->b_bcount, offset);
+	}
 	//if (ret == -1)
 	//	printf("ld: errno = %d\n", thunk_geterrno());
 	/* setup return params */
 	if ((ret >= 0) && (ret == bp->b_bcount)) {
 		bp->b_resid = 0;
 	} else {
 		bp->b_error = thunk_geterrno();
 		bp->b_resid = bp->b_bcount;
+	}