lkml.org 
[lkml]   [2022]   [Jan]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] mm/highmem: remove unnecessary done label
From
On 26.01.22 10:25, Miaohe Lin wrote:
> Remove unnecessary done label to simplify the code.
>
> Signed-off-by: Miaohe Lin <linmiaohe@huawei.com>
> ---
> mm/highmem.c | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/mm/highmem.c b/mm/highmem.c
> index 762679050c9a..0cc0c4da7ed9 100644
> --- a/mm/highmem.c
> +++ b/mm/highmem.c
> @@ -736,11 +736,11 @@ void *page_address(const struct page *page)
> list_for_each_entry(pam, &pas->lh, list) {
> if (pam->page == page) {
> ret = pam->virtual;
> - goto done;
> + break;
> }
> }
> }
> -done:
> +
> spin_unlock_irqrestore(&pas->lock, flags);
> return ret;
> }
> @@ -773,13 +773,12 @@ void set_page_address(struct page *page, void *virtual)
> list_for_each_entry(pam, &pas->lh, list) {
> if (pam->page == page) {
> list_del(&pam->list);
> - spin_unlock_irqrestore(&pas->lock, flags);
> - goto done;
> + break;
> }
> }
> spin_unlock_irqrestore(&pas->lock, flags);
> }
> -done:
> +
> return;
> }
>

Reviewed-by: David Hildenbrand <david@redhat.com>

--
Thanks,

David / dhildenb

\
 
 \ /
  Last update: 2022-01-26 11:22    [W:0.062 / U:0.040 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site