[Xen-devel,1/3] xen/arm: hvm_domain does not need to be cacheline aligned

Message ID 20170308180602.24430-2-julien.grall@arm.com
State New
Headers show
Series
  • xen/arm: p2m: Perform local invalidation on vCPU migration
Related show

Commit Message

Julien Grall March 8, 2017, 6:06 p.m.
hvm_domain only contains the HVM_PARAM that on ARM are not used often.
So it is not necessary to have hvm_domain fitting in a cacheline. Drop
it to save 128 bytes in the structure arch_domain.

Signed-off-by: Julien Grall <julien.grall@arm.com>
---
 xen/include/asm-arm/domain.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Stefano Stabellini March 8, 2017, 6:30 p.m. | #1
On Wed, 8 Mar 2017, Julien Grall wrote:
> hvm_domain only contains the HVM_PARAM that on ARM are not used often.
> So it is not necessary to have hvm_domain fitting in a cacheline. Drop
> it to save 128 bytes in the structure arch_domain.
> 
> Signed-off-by: Julien Grall <julien.grall@arm.com>

Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>


> ---
>  xen/include/asm-arm/domain.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/xen/include/asm-arm/domain.h b/xen/include/asm-arm/domain.h
> index f117e577f0..09fe502fc2 100644
> --- a/xen/include/asm-arm/domain.h
> +++ b/xen/include/asm-arm/domain.h
> @@ -14,7 +14,7 @@
>  struct hvm_domain
>  {
>      uint64_t              params[HVM_NR_PARAMS];
> -}  __cacheline_aligned;
> +};
>  
>  #ifdef CONFIG_ARM_64
>  enum domain_type {
> -- 
> 2.11.0
>

Patch hide | download patch | download mbox

diff --git a/xen/include/asm-arm/domain.h b/xen/include/asm-arm/domain.h
index f117e577f0..09fe502fc2 100644
--- a/xen/include/asm-arm/domain.h
+++ b/xen/include/asm-arm/domain.h
@@ -14,7 +14,7 @@ 
 struct hvm_domain
 {
     uint64_t              params[HVM_NR_PARAMS];
-}  __cacheline_aligned;
+};
 
 #ifdef CONFIG_ARM_64
 enum domain_type {