[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 5/6] nvdimm acpi: let qemu handle _DSM method
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH 5/6] nvdimm acpi: let qemu handle _DSM method |
Date: |
Fri, 8 Jan 2016 17:08:33 +0100 |
On Fri, 8 Jan 2016 12:01:54 +0800
Xiao Guangrong <address@hidden> wrote:
> On 01/07/2016 10:22 PM, Igor Mammedov wrote:
> > On Tue, 5 Jan 2016 02:52:07 +0800
> > Xiao Guangrong <address@hidden> wrote:
> >
> >> If dsm memory is successfully patched, we let qemu fully emulate
> >> the dsm method
> >>
> >> This patch saves _DSM input parameters into dsm memory, tell dsm
> >> memory address to QEMU, then fetch the result from the dsm memory
> > you also need to add NVDR._CRS method that would report
> > resources used by operation regions.
>
> I can not understand this point, why we need to report the resource
> of OperationRegion? It is ACPI internally used anyway.
so that OSPM could see that particular range is in use
and be able to notice conflicts if it happens some day.
>
> >
> > NVDIMM_COMMON_DSM - probably should be serialized, otherwise
> > there is a race risk, when several callers would write to
> > control region.
>
> Yes, i did it in patch 6/6, but definitely i should more it to here.
>
- Re: [Qemu-devel] [PATCH 3/6] nvdimm acpi: introduce patched dsm memory, (continued)
[Qemu-devel] [PATCH 1/6] pc: acpi: bump DSDT/SSDT compliance revision to v2, Xiao Guangrong, 2016/01/04
[Qemu-devel] [PATCH 2/6] nvdimm acpi: initialize the resource used by NVDIMM ACPI, Xiao Guangrong, 2016/01/04
[Qemu-devel] [PATCH 4/6] acpi: allow using acpi named offset for OperationRegion, Xiao Guangrong, 2016/01/04
[Qemu-devel] [PATCH 5/6] nvdimm acpi: let qemu handle _DSM method, Xiao Guangrong, 2016/01/04
[Qemu-devel] [PATCH 6/6] nvdimm acpi: emulate dsm method, Xiao Guangrong, 2016/01/04
Re: [Qemu-devel] [PATCH 0/6] NVDIMM ACPI: introduce the framework of QEMU emulated DSM, Igor Mammedov, 2016/01/07