On 7 March 2018 at 13:29, Thomas Huth <[email protected]> wrote: > On 07.03.2018 14:02, Paolo Bonzini wrote: >> Fixes: 11e732a5ed46903f997985bed4c3767ca28a7eb6 >> Reported-by: Cornelia Huck <[email protected]> >> Reported-by: luigi burdo <[email protected]> >> Signed-off-by: Paolo Bonzini <[email protected]> >> --- >> exec.c | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/exec.c b/exec.c >> index 604f03c535..a9181e6417 100644 >> --- a/exec.c >> +++ b/exec.c >> @@ -3393,7 +3393,6 @@ static bool flatview_access_valid(FlatView *fv, hwaddr >> addr, int len, >> if (!memory_access_is_direct(mr, is_write)) { >> l = memory_access_size(mr, l, addr); >> if (!memory_region_access_valid(mr, xlat, l, is_write)) { >> - rcu_read_unlock(); >> return false; >> } >> } >> > > Thanks, this seems to fix the problem with the sam640ex and 40p machines. > > Tested-by: Thomas Huth <[email protected]>
Thanks, applied to master since I ran into this too. -- PMM
