sergey.fedo...@linaro.org writes:
> From: Sergey Fedorov <serge.f...@gmail.com>
>
> tb_jmp_remove() was only used to remove the TB from a list of all TBs
> jumping to the same TB which is n-th jump destination of the given TB.
> Put a comment briefly describing the function behavior and rename it to
> better reflect its purpose.
>
> Signed-off-by: Sergey Fedorov <serge.f...@gmail.com>
> Signed-off-by: Sergey Fedorov <sergey.fedo...@linaro.org>
> ---
> translate-all.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/translate-all.c b/translate-all.c
> index f68716e1819f..28154ed75888 100644
> --- a/translate-all.c
> +++ b/translate-all.c
> @@ -922,7 +922,8 @@ static inline void tb_page_remove(TranslationBlock **ptb,
> TranslationBlock *tb)
> }
> }
>
> -static inline void tb_jmp_remove(TranslationBlock *tb, int n)
> +/* remove the TB from a list of TBs jumping to the n-th jump target of the
> TB */
> +static inline void tb_remove_from_jmp_list(TranslationBlock *tb, int n)
> {
> TranslationBlock *tb1;
> uintptr_t *ptb;
> @@ -995,8 +996,8 @@ void tb_phys_invalidate(TranslationBlock *tb,
> tb_page_addr_t page_addr)
> }
>
> /* suppress this TB from the two jump lists */
> - tb_jmp_remove(tb, 0);
> - tb_jmp_remove(tb, 1);
> + tb_remove_from_jmp_list(tb, 0);
> + tb_remove_from_jmp_list(tb, 1);
>
> /* suppress any remaining jumps to this TB */
> tb1 = tb->jmp_list_first;
Reviewed-by: Alex Bennée <alex.ben...@linaro.org>
--
Alex Bennée
