On Thu, Apr 30, 2015 at 12:29:30PM +1000, Alexey Kardashevskiy wrote:

On 04/29/2015 12:16 PM, David Gibson wrote:

quoted

On Sat, Apr 25, 2015 at 10:14:33PM +1000, Alexey Kardashevskiy wrote:

quoted

This is to make extended ownership and multiple groups support patches
simpler for review.

This should cause no behavioural change.

Um.. this doesn't appear to be true.  Previously removing a group from
an enabled container would fail with EBUSY, now it forces a disable.

This is the original tce_iommu_detach_group() where I cannot find EBUSY you
are referring to; it did and does enforce disable. What do I miss
here?

Sorry, my mistake.  I misread the patch.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson