[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v5 0/7] <asm/atomic.h> adjustments



On 7/15/2016 4:05 PM, Andrew Cooper wrote:
On 15/07/16 11:41, Corneliu ZUZU wrote:
Corneliu ZUZU (7):
   1/7: asm-arm/atomic.h: fix arm32|arm64 macros duplication
     Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
     Changed:
         * remove paragraph(s) from README.LinuxPrimitives file
         * no empty line additions

   2/7: asm-x86/atomic.h: minor: proper atomic_inc_and_test() placement
     Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
     Changed: <nothing>

   3/7: asm-arm/atomic.h: reorder macros to match x86-side
     Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
     Acked-by: Julien Grall <julien.grall@xxxxxxx>
     Changed: <nothing>

   4/7: asm/atomic.h: common prototyping (add xen/atomic.h)
     Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
     Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
     Acked-by: Julien Grall <julien.grall@xxxxxxx>
     Changed:
         * removed paragraph(s) in README.LinuxPrimitives file

   5/7: xen/atomic.h: fix: make atomic_read() param const
     Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
     Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
     Reviewed-by: Julien Grall <julien.grall@xxxxxxx>
     Changed: <nothing>

   6/7: asm-arm/atomic.h: atomic_{inc,dec}_return: macros to inline functions
     Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
     Changed:
         * README.LinuxPrimitives not updated anymore

   7/7: asm/atomic.h: implement missing and add common prototypes
     Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
     Acked-by: Julien Grall <julien.grall@xxxxxxx>
     Changed: <nothing>

  xen/arch/arm/README.LinuxPrimitives |   8 ++
  xen/include/asm-arm/arm32/atomic.h  |  13 ---
  xen/include/asm-arm/arm64/atomic.h  |  13 ---
  xen/include/asm-arm/atomic.h        |  59 ++++++++--
  xen/include/asm-x86/atomic.h        | 151 +++++++-------------------
  xen/include/xen/atomic.h            | 207 ++++++++++++++++++++++++++++++++++++
  6 files changed, 302 insertions(+), 149 deletions(-)
  create mode 100644 xen/include/xen/atomic.h
Committed.  Thankyou very much for untangling this rats nest.

~Andrew


Thanks too!
As an unrelated request, could you please provide your opinion on the question I've asked @ "[PATCH 03/16] x86/monitor: mechanical renames" ?
I'm not expecting a specific answer, I just need to know what to do next.

Thanks,
Corneliu.

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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.