[PATCH v2] drm/i915/gvt: Fix emulated vbt size issue

Zhenyu Wang zhenyuw at linux.intel.com
Fri Mar 6 01:11:57 UTC 2020


On 2020.03.05 21:15:59 +0800, Tina Zhang wrote:
> The emulated vbt doesn't tell its size correctly. According to the
> intel_vbt_defs.h, vbt_header.vbt_size should the size of VBT (VBT Header,
> BDB Header and data blocks), and bdb_header.bdb_size should be the size
> of BDB (BDB Header and data blocks).
> 
> This patch fixes the issue and lets vbt provided by GVT-g pass the guest
> i915's sanity test.
> 
> v2: refine the commit message. (Zhenyu)
> 
> Signed-off-by: Tina Zhang <tina.zhang at intel.com>
> ---
>  drivers/gpu/drm/i915/gvt/opregion.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/gvt/opregion.c b/drivers/gpu/drm/i915/gvt/opregion.c
> index 867e7629025b..33569b910ed5 100644
> --- a/drivers/gpu/drm/i915/gvt/opregion.c
> +++ b/drivers/gpu/drm/i915/gvt/opregion.c
> @@ -147,15 +147,14 @@ static void virt_vbt_generation(struct vbt *v)
>  	/* there's features depending on version! */
>  	v->header.version = 155;
>  	v->header.header_size = sizeof(v->header);
> -	v->header.vbt_size = sizeof(struct vbt) - sizeof(v->header);
> +	v->header.vbt_size = sizeof(struct vbt);
>  	v->header.bdb_offset = offsetof(struct vbt, bdb_header);
>  
>  	strcpy(&v->bdb_header.signature[0], "BIOS_DATA_BLOCK");
>  	v->bdb_header.version = 186; /* child_dev_size = 33 */
>  	v->bdb_header.header_size = sizeof(v->bdb_header);
>  
> -	v->bdb_header.bdb_size = sizeof(struct vbt) - sizeof(struct vbt_header)
> -		- sizeof(struct bdb_header);
> +	v->bdb_header.bdb_size = sizeof(struct vbt) - sizeof(struct vbt_header);
>  
>  	/* general features */
>  	v->general_features_header.id = BDB_GENERAL_FEATURES;
> -- 
> 2.17.1
> 

Reviewed-by: Zhenyu Wang <zhenyuw at linux.intel.com>

-- 
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/intel-gvt-dev/attachments/20200306/04a9e9cd/attachment.sig>


More information about the intel-gvt-dev mailing list