On Sat, 22 Aug 2020 at 16:13, Philippe Mathieu-Daudé <phi...@redhat.com> wrote:
>
> +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.
>
Phil, Thanks for the heads up on this !
No objections to removing this. We can easily re-add it if/when needed.
Thanks,
-Rob
