From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [PATCH v5 5/5] nvdimm acpi: add _CRS Date: Thu, 3 Mar 2016 16:48:55 +0200 Message-ID: <20160303164730-mutt-send-email-mst@redhat.com> References: <1456919441-101204-1-git-send-email-guangrong.xiao@linux.intel.com> <1456919441-101204-6-git-send-email-guangrong.xiao@linux.intel.com> <20160303152819-mutt-send-email-mst@redhat.com> <56D844AB.5060805@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: pbonzini@redhat.com, imammedo@redhat.com, gleb@kernel.org, mtosatti@redhat.com, stefanha@redhat.com, rth@twiddle.net, ehabkost@redhat.com, dan.j.williams@intel.com, kvm@vger.kernel.org, qemu-devel@nongnu.org To: Xiao Guangrong Return-path: Received: from mx1.redhat.com ([209.132.183.28]:45413 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757869AbcCCOtA (ORCPT ); Thu, 3 Mar 2016 09:49:00 -0500 Content-Disposition: inline In-Reply-To: <56D844AB.5060805@linux.intel.com> Sender: kvm-owner@vger.kernel.org List-ID: On Thu, Mar 03, 2016 at 10:05:31PM +0800, Xiao Guangrong wrote: > > > On 03/03/2016 09:29 PM, Michael S. Tsirkin wrote: > >On Wed, Mar 02, 2016 at 07:50:41PM +0800, Xiao Guangrong wrote: > >>As Igor suggested that we can report the BIOS patched operation region > >>so that OSPM could see that particular range is in use and be able to > >>notice conflicts if it happens some day > >> > >>Signed-off-by: Xiao Guangrong > > > >This is reserved RAM, exposing it in _CRS makes no sense to me. > > As more and more memory will be reserved by BIOS/QEMU, report the > information to OSPM and let it check the potential error is bad, > no? :) guest has enough info to detect conflicts if it wishes to. IIUC _CRS is not intended for RAM, it's for MMIO resources, if it works for RAM that's an accident. -- MST