Re: [PATCH v2 2/2] powerpc/nvdimm: use H_SCM_QUERY hcall on H_OVERLAP error

2019-09-24 Thread Aneesh Kumar K.V
Vaibhav Jain writes: >> +err_out: >> +dev_info(&p->pdev->dev, >> + "Failed to query, trying an unbind followed by bind"); >> +drc_pmem_unbind(p); >> +return drc_pmem_bind(p); >> +} > Would have preferred error handling for bind failure to be done at > single location i.e i

Re: [PATCH v2 2/2] powerpc/nvdimm: use H_SCM_QUERY hcall on H_OVERLAP error

2019-09-03 Thread Vaibhav Jain
Hi Aneesh, Thanks for the patch. A minor suggestion below: "Aneesh Kumar K.V" writes: > Right now we force an unbind of SCM memory at drcindex on H_OVERLAP error. > This really slows down operations like kexec where we get the H_OVERLAP > error because we don't go through a full hypervisor re i

[PATCH v2 2/2] powerpc/nvdimm: use H_SCM_QUERY hcall on H_OVERLAP error

2019-09-03 Thread Aneesh Kumar K.V
Right now we force an unbind of SCM memory at drcindex on H_OVERLAP error. This really slows down operations like kexec where we get the H_OVERLAP error because we don't go through a full hypervisor re init. H_OVERLAP error for a H_SCM_BIND_MEM hcall indicates that SCM memory at drc index is alrea