Re: arm: dmamap_destroy: remove explicit unload of map

2016-03-09 Thread David Gwynne
> On 6 Mar 2016, at 9:53 PM, Tobias Ulmer wrote: > > map is passed straight into free where it gets overwritten with junk. > No other arch makes map invalid before free, and my N2100 didn't > suddenly misbehave either. > > ok? ok > > Index: arch/arm/arm/bus_dma.c > ==

Re: arm: dmamap_destroy: remove explicit unload of map

2016-03-08 Thread Daniel Bolgheroni
On Sun, Mar 06, 2016 at 12:53:24PM +0100, Tobias Ulmer wrote: > map is passed straight into free where it gets overwritten with junk. > No other arch makes map invalid before free, and my N2100 didn't > suddenly misbehave either. > > ok? Have been building -current with this diff applied too and

Re: arm: dmamap_destroy: remove explicit unload of map

2016-03-06 Thread Patrick Wildt
On Sun, Mar 06, 2016 at 12:53:24PM +0100, Tobias Ulmer wrote: > map is passed straight into free where it gets overwritten with junk. > No other arch makes map invalid before free, and my N2100 didn't > suddenly misbehave either. Been building base with this diff, no issues. Also it doesn't seem

arm: dmamap_destroy: remove explicit unload of map

2016-03-06 Thread Tobias Ulmer
map is passed straight into free where it gets overwritten with junk. No other arch makes map invalid before free, and my N2100 didn't suddenly misbehave either. ok? Index: arch/arm/arm/bus_dma.c === RCS file: /home/vcs/cvs/openbsd/s