[PATCH] drm/i915/gvt: Fix firmware loading interface for GVT-g golden HW state
Zhenyu Wang
zhenyuw at linux.intel.com
Fri Mar 31 08:47:31 UTC 2017
On 2017.03.31 16:30:24 +0800, Zhi Wang wrote:
> Tested on my SKL, BTW: Zhenyuw: the script works well. :P I adjust the code
> to fit the name of file generated by the script.
>
oh, I think that script should fit for kernel instead, but not vice versa...
> For further usage of loading golden HW state, I think we need to prevent the
> user to reboot or reload the module after updating the golden HW state.
>
> Also, we need to bump up the firmware version after changing MMIO table as
> currently we only read HW mmio according to the MMIO table.
>
oh, my...that's horrible..
> Let's see how the requirement goes in future.
>
> Thanks,
> Zhi.
>
> On 04/01/17 00:00, Zhi Wang wrote:
> > Firmware loading interface for GVT-g golden HW state has been broken
> > before. This patch fixes GVT-g firmware loading interface. A user should
> > apply this patch if he wants to load GVT-g golden HW state from firmware
> > interface.
> >
> > Fixes: 579cea5 ("drm/i915/gvt: golden virtual HW state management")
> > Cc: Zhenyu Wang <zhenyuw at linux.intel.com>
> > Cc: drm-intel-fixes at lists.freedesktop.org
> > Signed-off-by: Zhi Wang <zhi.a.wang at intel.com>
> > ---
> > drivers/gpu/drm/i915/gvt/firmware.c | 9 ++++++---
> > 1 file changed, 6 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/gvt/firmware.c b/drivers/gpu/drm/i915/gvt/firmware.c
> > index 933a7c2..dce8d15 100644
> > --- a/drivers/gpu/drm/i915/gvt/firmware.c
> > +++ b/drivers/gpu/drm/i915/gvt/firmware.c
> > @@ -75,11 +75,11 @@ static int expose_firmware_sysfs(struct intel_gvt *gvt)
> > struct gvt_firmware_header *h;
> > void *firmware;
> > void *p;
> > - unsigned long size;
> > + unsigned long size, crc32_start;
> > int i;
> > int ret;
> >
> > - size = sizeof(*h) + info->mmio_size + info->cfg_space_size - 1;
> > + size = sizeof(*h) + info->mmio_size + info->cfg_space_size;
> > firmware = vzalloc(size);
> > if (!firmware)
> > return -ENOMEM;
> > @@ -112,6 +112,9 @@ static int expose_firmware_sysfs(struct intel_gvt *gvt)
> >
> > memcpy(gvt->firmware.mmio, p, info->mmio_size);
> >
> > + crc32_start = offsetof(struct gvt_firmware_header, crc32) + 4;
> > + h->crc32 = crc32_le(0, firmware + crc32_start, size - crc32_start);
> > +
> > firmware_attr.size = size;
> > firmware_attr.private = firmware;
> >
> > @@ -234,7 +237,7 @@ int intel_gvt_load_firmware(struct intel_gvt *gvt)
> >
> > firmware->mmio = mem;
> >
> > - sprintf(path, "%s/vid_0x%04x_did_0x%04x_rid_0x%04x.golden_hw_state",
> > + sprintf(path, "%s/vid_0x%04x_did_0x%04x_rid_0x%02x.golden_hw_state",
> > GVT_FIRMWARE_PATH, pdev->vendor, pdev->device,
> > pdev->revision);
> >
> >
--
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: 163 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/intel-gvt-dev/attachments/20170331/a06d329a/attachment.sig>
More information about the intel-gvt-dev
mailing list