public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Gao, Liming" <liming.gao@intel.com>
To: Leif Lindholm <leif.lindholm@linaro.org>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Kinney, Michael D" <michael.d.kinney@intel.com>,
	Ard Biesheuvel <ard.biesheuvel@linaro.org>
Subject: Re: [PATCH] MdePkg: add ARM/AARCH64 support to BaseCacheMaintenanceLib
Date: Wed, 20 Sep 2017 14:45:30 +0000	[thread overview]
Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14E15B35B@SHSMSX152.ccr.corp.intel.com> (raw)
In-Reply-To: <20170920132252.11761-1-leif.lindholm@linaro.org>

Leif:
  This change lets MdePkg BaseCacheMaintenanceLib depend on ArmPkg ArmLib. But, MdePkg is the basic package. It should not depend on other package. I suggest to add this ARM specific BaseCacheMaintenanceLib library instance into ArmPkg. 

Thanks	
Liming
> -----Original Message-----
> From: Leif Lindholm [mailto:leif.lindholm@linaro.org]
> Sent: Wednesday, September 20, 2017 9:23 PM
> To: edk2-devel@lists.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com>; Ard Biesheuvel
> <ard.biesheuvel@linaro.org>
> Subject: [PATCH] MdePkg: add ARM/AARCH64 support to BaseCacheMaintenanceLib
> 
> ARM platforms have been using a separately located library in ArmPkg for
> high-level cache maintenance calls. Resolve this anomaly by overwriting
> ArmCache.c with the contents of
> ArmPkg/Library/ArmCacheMaintenanceLib/ArmCacheMaintenanceLib.c, and add
> the ArmLib dependency for the affected architectures.
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
> 
> The intent is to delete the ArmPkg version once no upstream platforms
> are using it.
> 
>  MdePkg/Library/BaseCacheMaintenanceLib/ArmCache.c  | 222 +++++----------------
>  .../BaseCacheMaintenanceLib.inf                    |   2 +
>  2 files changed, 55 insertions(+), 169 deletions(-)
> 
> diff --git a/MdePkg/Library/BaseCacheMaintenanceLib/ArmCache.c b/MdePkg/Library/BaseCacheMaintenanceLib/ArmCache.c
> index 79c84a0982..0759e38cd4 100644
> --- a/MdePkg/Library/BaseCacheMaintenanceLib/ArmCache.c
> +++ b/MdePkg/Library/BaseCacheMaintenanceLib/ArmCache.c
> @@ -1,67 +1,63 @@
>  /** @file
> -  Cache Maintenance Functions. These functions vary by ARM architecture so the MdePkg
> -  versions are null functions used to make sure things will compile.
> 
> -  Copyright (c) 2006 - 2009, Intel Corporation. All rights reserved.<BR>
> -  Portions copyright (c) 2008 - 2009, Apple Inc. All rights reserved.<BR>
> +  Copyright (c) 2008 - 2009, Apple Inc. All rights reserved.<BR>
> +  Copyright (c) 2011 - 2014, ARM Limited. All rights reserved.
> +
>    This program and the accompanying materials
>    are licensed and made available under the terms and conditions of the BSD License
>    which accompanies this distribution.  The full text of the license may be found at
> -  http://opensource.org/licenses/bsd-license.php.
> +  http://opensource.org/licenses/bsd-license.php
> 
>    THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
>    WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> 
>  **/
> -
> -//
> -// Include common header file for this module.
> -//
>  #include <Base.h>
> +#include <Library/ArmLib.h>
>  #include <Library/DebugLib.h>
> +#include <Library/PcdLib.h>
> 
> -/**
> -  Invalidates the entire instruction cache in cache coherency domain of the
> -  calling CPU.
> -
> -  Invalidates the entire instruction cache in cache coherency domain of the
> -  calling CPU.
> +STATIC
> +VOID
> +CacheRangeOperation (
> +  IN  VOID            *Start,
> +  IN  UINTN           Length,
> +  IN  LINE_OPERATION  LineOperation,
> +  IN  UINTN           LineLength
> +  )
> +{
> +  UINTN ArmCacheLineAlignmentMask  = LineLength - 1;
> +
> +  // Align address (rounding down)
> +  UINTN AlignedAddress = (UINTN)Start - ((UINTN)Start & ArmCacheLineAlignmentMask);
> +  UINTN EndAddress     = (UINTN)Start + Length;
> +
> +  // Perform the line operation on an address in each cache line
> +  while (AlignedAddress < EndAddress) {
> +    LineOperation(AlignedAddress);
> +    AlignedAddress += LineLength;
> +  }
> +  ArmDataSynchronizationBarrier ();
> +}
> 
> -**/
>  VOID
>  EFIAPI
>  InvalidateInstructionCache (
>    VOID
>    )
>  {
> -  ASSERT(FALSE);
> +  ASSERT (FALSE);
>  }
> 
> -/**
> -  Invalidates a range of instruction cache lines in the cache coherency domain
> -  of the calling CPU.
> -
> -  Invalidates the instruction cache lines specified by Address and Length. If
> -  Address is not aligned on a cache line boundary, then entire instruction
> -  cache line containing Address is invalidated. If Address + Length is not
> -  aligned on a cache line boundary, then the entire instruction cache line
> -  containing Address + Length -1 is invalidated. This function may choose to
> -  invalidate the entire instruction cache if that is more efficient than
> -  invalidating the specified range. If Length is 0, then no instruction cache
> -  lines are invalidated. Address is returned.
> -
> -  If Length is greater than (MAX_ADDRESS - Address + 1), then ASSERT().
> -
> -  @param  Address The base address of the instruction cache lines to
> -                  invalidate. If the CPU is in a physical addressing mode, then
> -                  Address is a physical address. If the CPU is in a virtual
> -                  addressing mode, then Address is a virtual address.
> -
> -  @param  Length  The number of bytes to invalidate from the instruction cache.
> -
> -  @return Address
> +VOID
> +EFIAPI
> +InvalidateDataCache (
> +  VOID
> +  )
> +{
> +  ASSERT (FALSE);
> +}
> 
> -**/
>  VOID *
>  EFIAPI
>  InvalidateInstructionCacheRange (
> @@ -69,56 +65,26 @@ InvalidateInstructionCacheRange (
>    IN      UINTN                     Length
>    )
>  {
> -  ASSERT (Length <= MAX_ADDRESS - (UINTN)Address + 1);
> -  ASSERT(FALSE);
> -  return Address;
> -}
> +  CacheRangeOperation (Address, Length, ArmCleanDataCacheEntryToPoUByMVA,
> +    ArmDataCacheLineLength ());
> +  CacheRangeOperation (Address, Length,
> +    ArmInvalidateInstructionCacheEntryToPoUByMVA,
> +    ArmInstructionCacheLineLength ());
> 
> -/**
> -  Writes back and invalidates the entire data cache in cache coherency domain
> -  of the calling CPU.
> +  ArmInstructionSynchronizationBarrier ();
> 
> -  Writes Back and Invalidates the entire data cache in cache coherency domain
> -  of the calling CPU. This function guarantees that all dirty cache lines are
> -  written back to system memory, and also invalidates all the data cache lines
> -  in the cache coherency domain of the calling CPU.
> +  return Address;
> +}
> 
> -**/
>  VOID
>  EFIAPI
>  WriteBackInvalidateDataCache (
>    VOID
>    )
>  {
> -  ASSERT(FALSE);
> +  ASSERT (FALSE);
>  }
> 
> -/**
> -  Writes back and invalidates a range of data cache lines in the cache
> -  coherency domain of the calling CPU.
> -
> -  Writes back and invalidates the data cache lines specified by Address and
> -  Length. If Address is not aligned on a cache line boundary, then entire data
> -  cache line containing Address is written back and invalidated. If Address +
> -  Length is not aligned on a cache line boundary, then the entire data cache
> -  line containing Address + Length -1 is written back and invalidated. This
> -  function may choose to write back and invalidate the entire data cache if
> -  that is more efficient than writing back and invalidating the specified
> -  range. If Length is 0, then no data cache lines are written back and
> -  invalidated. Address is returned.
> -
> -  If Length is greater than (MAX_ADDRESS - Address + 1), then ASSERT().
> -
> -  @param  Address The base address of the data cache lines to write back and
> -                  invalidate. If the CPU is in a physical addressing mode, then
> -                  Address is a physical address. If the CPU is in a virtual
> -                  addressing mode, then Address is a virtual address.
> -  @param  Length  The number of bytes to write back and invalidate from the
> -                  data cache.
> -
> -  @return Address
> -
> -**/
>  VOID *
>  EFIAPI
>  WriteBackInvalidateDataCacheRange (
> @@ -126,55 +92,20 @@ WriteBackInvalidateDataCacheRange (
>    IN      UINTN                     Length
>    )
>  {
> -  ASSERT (Length <= MAX_ADDRESS - (UINTN)Address + 1);
> -  ASSERT(FALSE);
> +  CacheRangeOperation(Address, Length, ArmCleanInvalidateDataCacheEntryByMVA,
> +    ArmDataCacheLineLength ());
>    return Address;
>  }
> 
> -/**
> -  Writes back the entire data cache in cache coherency domain of the calling
> -  CPU.
> -
> -  Writes back the entire data cache in cache coherency domain of the calling
> -  CPU. This function guarantees that all dirty cache lines are written back to
> -  system memory. This function may also invalidate all the data cache lines in
> -  the cache coherency domain of the calling CPU.
> -
> -**/
>  VOID
>  EFIAPI
>  WriteBackDataCache (
>    VOID
>    )
>  {
> -  ASSERT(FALSE);
> +  ASSERT (FALSE);
>  }
> 
> -/**
> -  Writes back a range of data cache lines in the cache coherency domain of the
> -  calling CPU.
> -
> -  Writes back the data cache lines specified by Address and Length. If Address
> -  is not aligned on a cache line boundary, then entire data cache line
> -  containing Address is written back. If Address + Length is not aligned on a
> -  cache line boundary, then the entire data cache line containing Address +
> -  Length -1 is written back. This function may choose to write back the entire
> -  data cache if that is more efficient than writing back the specified range.
> -  If Length is 0, then no data cache lines are written back. This function may
> -  also invalidate all the data cache lines in the specified range of the cache
> -  coherency domain of the calling CPU. Address is returned.
> -
> -  If Length is greater than (MAX_ADDRESS - Address + 1), then ASSERT().
> -
> -  @param  Address The base address of the data cache lines to write back. If
> -                  the CPU is in a physical addressing mode, then Address is a
> -                  physical address. If the CPU is in a virtual addressing
> -                  mode, then Address is a virtual address.
> -  @param  Length  The number of bytes to write back from the data cache.
> -
> -  @return Address
> -
> -**/
>  VOID *
>  EFIAPI
>  WriteBackDataCacheRange (
> @@ -182,58 +113,11 @@ WriteBackDataCacheRange (
>    IN      UINTN                     Length
>    )
>  {
> -  ASSERT (Length <= MAX_ADDRESS - (UINTN)Address + 1);
> -  ASSERT(FALSE);
> +  CacheRangeOperation(Address, Length, ArmCleanDataCacheEntryByMVA,
> +    ArmDataCacheLineLength ());
>    return Address;
>  }
> 
> -/**
> -  Invalidates the entire data cache in cache coherency domain of the calling
> -  CPU.
> -
> -  Invalidates the entire data cache in cache coherency domain of the calling
> -  CPU. This function must be used with care because dirty cache lines are not
> -  written back to system memory. It is typically used for cache diagnostics. If
> -  the CPU does not support invalidation of the entire data cache, then a write
> -  back and invalidate operation should be performed on the entire data cache.
> -
> -**/
> -VOID
> -EFIAPI
> -InvalidateDataCache (
> -  VOID
> -  )
> -{
> -  ASSERT(FALSE);
> -}
> -
> -/**
> -  Invalidates a range of data cache lines in the cache coherency domain of the
> -  calling CPU.
> -
> -  Invalidates the data cache lines specified by Address and Length. If Address
> -  is not aligned on a cache line boundary, then entire data cache line
> -  containing Address is invalidated. If Address + Length is not aligned on a
> -  cache line boundary, then the entire data cache line containing Address +
> -  Length -1 is invalidated. This function must never invalidate any cache lines
> -  outside the specified range. If Length is 0, then no data cache lines are
> -  invalidated. Address is returned. This function must be used with care
> -  because dirty cache lines are not written back to system memory. It is
> -  typically used for cache diagnostics. If the CPU does not support
> -  invalidation of a data cache range, then a write back and invalidate
> -  operation should be performed on the data cache range.
> -
> -  If Length is greater than (MAX_ADDRESS - Address + 1), then ASSERT().
> -
> -  @param  Address The base address of the data cache lines to invalidate. If
> -                  the CPU is in a physical addressing mode, then Address is a
> -                  physical address. If the CPU is in a virtual addressing mode,
> -                  then Address is a virtual address.
> -  @param  Length  The number of bytes to invalidate from the data cache.
> -
> -  @return Address
> -
> -**/
>  VOID *
>  EFIAPI
>  InvalidateDataCacheRange (
> @@ -241,7 +125,7 @@ InvalidateDataCacheRange (
>    IN      UINTN                     Length
>    )
>  {
> -  ASSERT (Length <= MAX_ADDRESS - (UINTN)Address + 1);
> -  ASSERT(FALSE);
> +  CacheRangeOperation(Address, Length, ArmInvalidateDataCacheEntryByMVA,
> +    ArmDataCacheLineLength ());
>    return Address;
>  }
> diff --git a/MdePkg/Library/BaseCacheMaintenanceLib/BaseCacheMaintenanceLib.inf
> b/MdePkg/Library/BaseCacheMaintenanceLib/BaseCacheMaintenanceLib.inf
> index d659161f33..7440a0062b 100644
> --- a/MdePkg/Library/BaseCacheMaintenanceLib/BaseCacheMaintenanceLib.inf
> +++ b/MdePkg/Library/BaseCacheMaintenanceLib/BaseCacheMaintenanceLib.inf
> @@ -59,3 +59,5 @@
>  [LibraryClasses.Ipf]
>    PalLib
> 
> +[LibraryClasses.ARM,LibraryClasses.AARCH64]
> +  ArmLib
> --
> 2.11.0



  reply	other threads:[~2017-09-20 14:42 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-20 13:22 [PATCH] MdePkg: add ARM/AARCH64 support to BaseCacheMaintenanceLib Leif Lindholm
2017-09-20 14:45 ` Gao, Liming [this message]
2017-09-20 15:05   ` Leif Lindholm
2017-09-20 17:31     ` Kinney, Michael D
2017-09-21  2:23       ` Gao, Liming
2017-10-06 15:26         ` Leif Lindholm

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A89E2EF3DFEDB4C8BFDE51014F606A14E15B35B@SHSMSX152.ccr.corp.intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox