On Tue, Sep 20, 2016 at 04:21:41PM +0300, David Kiarie wrote:
>     >     > +amdvi_mode_invalid(unsigned level, uint64_t addr)"error:
>     translation
>     >     level 0x%"PRIu8" translating addr 0x%"PRIx64
>     This is also wrong. Can you pls go over the traces?
> I think you might be looking the wrong patch here.

OK, must be the HTML in your mail messing things up.
Please post plain-text patches and I'll review.

>     Also, you can check Peter Maydell's rejects of the pulls that
>     included these patches. 
> I did look at the reject. The initial code wasn't actually broken but that
> discussion wasn't really relevant since 'extract64/extract32' is more decent.

There were some failures around traces too though. Pls check them out.


Reply via email to