Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 16 Oct 2013 16:35:25 +0000 (UTC)
From:      Ian Lepore <ian@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r256638 - head/sys/arm/arm
Message-ID:  <201310161635.r9GGZPkG032205@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ian
Date: Wed Oct 16 16:35:25 2013
New Revision: 256638
URL: http://svnweb.freebsd.org/changeset/base/256638

Log:
  Add cases for the combinations of busdma sync op flags that we handle
  correctly by doing nothing, then add a panic for the default case, because
  that implies that some driver asked for a sync (probably incorrectly) and
  nothing was done.

Modified:
  head/sys/arm/arm/busdma_machdep-v6.c

Modified: head/sys/arm/arm/busdma_machdep-v6.c
==============================================================================
--- head/sys/arm/arm/busdma_machdep-v6.c	Wed Oct 16 16:32:35 2013	(r256637)
+++ head/sys/arm/arm/busdma_machdep-v6.c	Wed Oct 16 16:35:25 2013	(r256638)
@@ -1282,7 +1282,12 @@ _bus_dmamap_sync(bus_dma_tag_t dmat, bus
 			}
 			break;
 
+		case BUS_DMASYNC_POSTREAD:
+		case BUS_DMASYNC_POSTWRITE:
+		case BUS_DMASYNC_POSTREAD | BUS_DMASYNC_POSTWRITE:
+			break;
 		default:
+			panic("unsupported combination of sync operations: 0x%08x\n", op);
 			break;
 		}
 	}



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