Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 13 Dec 2013 16:41:04 +0000 (UTC)
From:      Ian Lepore <ian@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org
Subject:   svn commit: r259310 - stable/10/sys/arm/arm
Message-ID:  <201312131641.rBDGf49Y026768@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ian
Date: Fri Dec 13 16:41:04 2013
New Revision: 259310
URL: http://svnweb.freebsd.org/changeset/base/259310

Log:
  MFC r256638:
  
    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:
  stable/10/sys/arm/arm/busdma_machdep-v6.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/arm/arm/busdma_machdep-v6.c
==============================================================================
--- stable/10/sys/arm/arm/busdma_machdep-v6.c	Fri Dec 13 16:38:21 2013	(r259309)
+++ stable/10/sys/arm/arm/busdma_machdep-v6.c	Fri Dec 13 16:41:04 2013	(r259310)
@@ -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?201312131641.rBDGf49Y026768>