[PATCH 9/9] drm/i915/gvt: Define PTE addr mask with GENMASK_ULL
Zhi Wang
zhi.a.wang at intel.com
Mon Dec 25 11:21:15 UTC 2017
This one looks nice.
BTW: GTT_HAW can be different on different SKUs.
On 12/25/17 17:11, changbin.du at intel.com wrote:
> From: Changbin Du <changbin.du at intel.com>
>
> Define the masks better.
>
> Signed-off-by: Changbin Du <changbin.du at intel.com>
> ---
> drivers/gpu/drm/i915/gvt/gtt.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gvt/gtt.c b/drivers/gpu/drm/i915/gvt/gtt.c
> index 1ce6393..5754884 100644
> --- a/drivers/gpu/drm/i915/gvt/gtt.c
> +++ b/drivers/gpu/drm/i915/gvt/gtt.c
> @@ -337,9 +337,9 @@ static inline int gtt_set_entry64(void *pt,
>
> #define GTT_HAW 46
>
> -#define ADDR_1G_MASK (((1UL << (GTT_HAW - 30)) - 1) << 30)
> -#define ADDR_2M_MASK (((1UL << (GTT_HAW - 21)) - 1) << 21)
> -#define ADDR_4K_MASK (((1UL << (GTT_HAW - 12)) - 1) << 12)
> +#define ADDR_1G_MASK GENMASK_ULL(GTT_HAW - 1, 30)
> +#define ADDR_2M_MASK GENMASK_ULL(GTT_HAW - 1, 21)
> +#define ADDR_4K_MASK GENMASK_ULL(GTT_HAW - 1, 12)
>
> static unsigned long gen8_gtt_get_pfn(struct intel_gvt_gtt_entry *e)
> {
>
More information about the intel-gvt-dev
mailing list