> From: ankita@xxxxxxxxxx <ankita@xxxxxxxxxx>
> Sent: Saturday, January 18, 2025 7:37 AM
> @@ -780,23 +787,31 @@ nvgrace_gpu_init_nvdev_struct(struct pci_dev
> *pdev,
> * memory (usemem) is added to the kernel for usage by the VM
> * workloads. Make the usable memory size memblock aligned.
> */
> - if (check_sub_overflow(memlength, RESMEM_SIZE,
> + if (check_sub_overflow(memlength, resmem_size,
> &nvdev->usemem.memlength)) {
> ret = -EOVERFLOW;
> goto done;
> }
>
> - /*
> - * The USEMEM part of the device memory has to be MEMBLK_SIZE
> - * aligned. This is a hardwired ABI value between the GPU FW and
> - * VFIO driver. The VM device driver is also aware of it and make
> - * use of the value for its calculation to determine USEMEM size.
> - */
> - nvdev->usemem.memlength = round_down(nvdev-
> >usemem.memlength,
> - MEMBLK_SIZE);
> - if (nvdev->usemem.memlength == 0) {
> - ret = -EINVAL;
> - goto done;
> + if (!nvdev->has_mig_hw_bug_fix) {
> + /*
> + * If the device memory is split to workaround the MIG bug,
> + * the USEMEM part of the device memory has to be
> MEMBLK_SIZE
> + * aligned. This is a hardwired ABI value between the GPU FW
> and
> + * VFIO driver. The VM device driver is also aware of it and
> make
> + * use of the value for its calculation to determine USEMEM
> size.
> + * Note that the device memory may not be 512M aligned.
> + *
> + * If the hardware has the fix for MIG, there is no
> requirement
> + * for splitting the device memory to create RESMEM. The
> entire
> + * device memory is usable and will be USEMEM.
Just double confirm. With the fix it's not required to have the usemem
512M aligned, or does hardware guarantee that usemem is always
512M aligned?
And it's clearer to return early when the fix is there so the majority of
the existing code can be left intact instead of causing unnecessary
indent here.
