[Xen-devel] Ping: [PATCH] add page_get_owner_and_reference() related ASSERT()s

["Jan Beulich" <JBeulich@xxxxxxxx>]

Folks on the To list: Ping?

Wei: Now that all patches to go in need your ack, I'm Cc-ing you.

Thanks, Jan

>>> On 24.07.15 at 14:37, <JBeulich@xxxxxxxx> wrote:
> The function shouldn't return NULL after having obtained a reference,
> or else the caller won't know to drop it.
> 
> Also its result shouldn't be ignored - if calling code is certain that
> a page already has a non-zero refcount, it better ASSERT()s so.
> 
> Finally this as well as get_page() and put_page() are required to be
> available on all architectures - move the declarations to xen/mm.h.
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> 
> --- a/xen/arch/arm/guestcopy.c
> +++ b/xen/arch/arm/guestcopy.c
> @@ -1,10 +1,8 @@
> -#include <xen/config.h>
>  #include <xen/lib.h>
>  #include <xen/domain_page.h>
> +#include <xen/mm.h>
>  #include <xen/sched.h>
>  #include <asm/current.h>
> -
> -#include <asm/mm.h>
>  #include <asm/guest_access.h>
>  
>  static unsigned long raw_copy_to_guest_helper(void *to, const void *from,
> --- a/xen/arch/arm/mm.c
> +++ b/xen/arch/arm/mm.c
> @@ -1170,6 +1170,7 @@ long arch_memory_op(int op, XEN_GUEST_HA
>  struct domain *page_get_owner_and_reference(struct page_info *page)
>  {
>      unsigned long x, y = page->count_info;
> +    struct domain *owner;
>  
>      do {
>          x = y;
> @@ -1182,7 +1183,10 @@ struct domain *page_get_owner_and_refere
>      }
>      while ( (y = cmpxchg(&page->count_info, x, x + 1)) != x );
>  
> -    return page_get_owner(page);
> +    owner = page_get_owner(page);
> +    ASSERT(owner);
> +
> +    return owner;
>  }
>  
>  void put_page(struct page_info *page)
> --- a/xen/arch/x86/mm.c
> +++ b/xen/arch/x86/mm.c
> @@ -2039,6 +2039,7 @@ void put_page(struct page_info *page)
>  struct domain *page_get_owner_and_reference(struct page_info *page)
>  {
>      unsigned long x, y = page->count_info;
> +    struct domain *owner;
>  
>      do {
>          x = y;
> @@ -2052,7 +2053,10 @@ struct domain *page_get_owner_and_refere
>      }
>      while ( (y = cmpxchg(&page->count_info, x, x + 1)) != x );
>  
> -    return page_get_owner(page);
> +    owner = page_get_owner(page);
> +    ASSERT(owner);
> +
> +    return owner;
>  }
>  
>  
> --- a/xen/common/grant_table.c
> +++ b/xen/common/grant_table.c
> @@ -2245,7 +2245,8 @@ __acquire_grant_for_copy(
>      {
>          ASSERT(mfn_valid(act->frame));
>          *page = mfn_to_page(act->frame);
> -        (void)page_get_owner_and_reference(*page);
> +        td = page_get_owner_and_reference(*page);
> +        ASSERT(td);
>      }
>  
>      act->pin += readonly ? GNTPIN_hstr_inc : GNTPIN_hstw_inc;
> --- a/xen/include/asm-arm/mm.h
> +++ b/xen/include/asm-arm/mm.h
> @@ -275,10 +275,6 @@ static inline void *page_to_virt(const s
>      return mfn_to_virt(page_to_mfn(pg));
>  }
>  
> -struct domain *page_get_owner_and_reference(struct page_info *page);
> -void put_page(struct page_info *page);
> -int  get_page(struct page_info *page, struct domain *domain);
> -
>  struct page_info *get_page_from_gva(struct domain *d, vaddr_t va,
>                                      unsigned long flags);
>  
> --- a/xen/include/asm-x86/mm.h
> +++ b/xen/include/asm-x86/mm.h
> @@ -352,9 +352,6 @@ const unsigned long *get_platform_badpag
>  int page_lock(struct page_info *page);
>  void page_unlock(struct page_info *page);
>  
> -struct domain *page_get_owner_and_reference(struct page_info *page);
> -void put_page(struct page_info *page);
> -int  get_page(struct page_info *page, struct domain *domain);
>  void put_page_type(struct page_info *page);
>  int  get_page_type(struct page_info *page, unsigned long type);
>  int  put_page_type_preemptible(struct page_info *page);
> --- a/xen/include/xen/mm.h
> +++ b/xen/include/xen/mm.h
> @@ -45,6 +45,7 @@
>  #ifndef __XEN_MM_H__
>  #define __XEN_MM_H__
>  
> +#include <xen/compiler.h>
>  #include <xen/types.h>
>  #include <xen/list.h>
>  #include <xen/spinlock.h>
> @@ -77,9 +78,12 @@ TYPE_SAFE(unsigned long, pfn);
>  #undef pfn_t
>  #endif
>  
> -struct domain;
>  struct page_info;
>  
> +struct domain *__must_check page_get_owner_and_reference(struct page_info 
> *);
> +void put_page(struct page_info *);
> +int get_page(struct page_info *, struct domain *);
> +
>  /* Boot-time allocator. Turns into generic allocator after bootstrap. */
>  void init_boot_pages(paddr_t ps, paddr_t pe);
>  unsigned long alloc_boot_pages(



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel