Am 2020-08-26 um 4:29 a.m. schrieb Hanjun Guo: > If the ignore_crat is set to non-zero value, it's no point getting > the CRAT table, so just move the ignore_crat check before we get the > CRAT table. > > Signed-off-by: Hanjun Guo <guohan...@huawei.com> > --- > drivers/gpu/drm/amd/amdkfd/kfd_crat.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_crat.c > b/drivers/gpu/drm/amd/amdkfd/kfd_crat.c > index 6a250f8..ed77b109 100644 > --- a/drivers/gpu/drm/amd/amdkfd/kfd_crat.c > +++ b/drivers/gpu/drm/amd/amdkfd/kfd_crat.c > @@ -764,6 +764,11 @@ int kfd_create_crat_image_acpi(void **crat_image, size_t > *size) > > *crat_image = NULL; > > + if (ignore_crat) {
A conflicting change in this area was just submitted on Monday to amd-staging-drm-next. You'll need to rebase your patch. It should be straight-forward. ignore_crat was replaced with a function call kfd_ignore_crat(). Other than that, your patch series looks good to me. If I don't see an update from you in a day or two, I'll fix it up myself and add my R-b. Thanks, Felix > + pr_info("CRAT table disabled by module option\n"); > + return -ENODATA; > + } > + > /* Fetch the CRAT table from ACPI */ > status = acpi_get_table(CRAT_SIGNATURE, 0, &crat_table); > if (status == AE_NOT_FOUND) { > @@ -776,11 +781,6 @@ int kfd_create_crat_image_acpi(void **crat_image, size_t > *size) > return -EINVAL; > } > > - if (ignore_crat) { > - pr_info("CRAT table disabled by module option\n"); > - return -ENODATA; > - } > - > pcrat_image = kmemdup(crat_table, crat_table->length, GFP_KERNEL); > if (!pcrat_image) > return -ENOMEM;