+Robert
On 8/7/20 12:16 PM, Paolo Bonzini wrote:
> On 07/08/20 12:02, Stefan Hajnoczi wrote:
>> On Thu, Aug 06, 2020 at 05:07:26PM +0200, Philippe Mathieu-Daudé wrote:
>>> Last uses of memory_region_clear_global_locking() have been
>>> removed in commit 7070e085d4 ("acpi: mark PMTIMER as unlocked")
>>> and commit 08565552f7 ("cputlb: Move NOTDIRTY handling from I/O
>>> path to TLB path").
>>> Remove memory_region_clear_global_locking() and the now unused
>>> 'global_locking' field in MemoryRegion.
>>>
>>> Reported-by: Alexander Bulekov <alx...@bu.edu>
>>> Suggested-by: Stefan Hajnoczi <stefa...@redhat.com>
>>> Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com>
>>> ---
>>> include/exec/memory.h | 14 --------------
>>> accel/tcg/cputlb.c | 4 ++--
>>> exec.c | 2 +-
>>> softmmu/memory.c | 6 ------
>>> 4 files changed, 3 insertions(+), 23 deletions(-)
>>
>> It can be added back in later, if necessary. For now let's drop the dead
>> code.
>>
>> Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com>
>>
>
> I expect it will come back since Linaro is working on BQL-free interrupt
> handling for TCG, but no objections.
Robert, any comment on this patch?
Thanks,
Phil.
