Re: [Qemu-devel] [PATCH 2/2] memory: Drop FlatRange.romd_mode

[Paolo Bonzini <pbonzini@redhat.com>]

On 24/05/2016 19:47, Laszlo Ersek wrote:
> Which I think satisfies (a->mr == b->mr), but falsifies (a->romd_mode
> == b->romd_mode).
> 
> In effect, the patch seems to allow merging and equality between
> FlatRange objects when they only differ in romd_mode, and that's
> wrong.
> 
> Given that the cover letter for this series says "memory: Dead code
> removals", I'm requesting that this patch be simply reverted.
> 
> Drew and myself bisected this independently, in parallel. The bug was
> originally reported by Drew, so I think on the revert commit, the
> Reported-by should belong to him. Beyond the bisection, Drew also
> tested the exact revert, and it restores functionality. For which
> reason, I'm proposing, for the revert patch:
> 
> Reported-by: Drew Jones <drjones@redhat.com>
> Tested-by: Drew Jones <drjones@redhat.com>
> 
> If my analysis above is correct, then I wouldn't mind taking credit
> for it, something like:
> 
> Analyzed-by: Laszlo Ersek <lersek@redhat.com>

That makes a lot of sense.  Revert on the way.

Thanks,

Paolo