From mboxrd@z Thu Jan 1 00:00:00 1970 From: Xiao Guangrong Subject: Re: [PATCH v5 2/5] nvdimm acpi: introduce patched dsm memory Date: Thu, 3 Mar 2016 21:35:57 +0800 Message-ID: <56D83DBD.2050401@linux.intel.com> References: <1456919441-101204-1-git-send-email-guangrong.xiao@linux.intel.com> <1456919441-101204-3-git-send-email-guangrong.xiao@linux.intel.com> <20160303151206-mutt-send-email-mst@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit 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: "Michael S. Tsirkin" Return-path: Received: from mga04.intel.com ([192.55.52.120]:42265 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751383AbcCCNgN (ORCPT ); Thu, 3 Mar 2016 08:36:13 -0500 In-Reply-To: <20160303151206-mutt-send-email-mst@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On 03/03/2016 09:12 PM, Michael S. Tsirkin wrote: >> - (void *)(table_data->data + table_data->len - ssdt->buf->len), >> - "SSDT", ssdt->buf->len, 1, NULL, "NVDIMM"); >> + (void *)(table_data->data + nvdimm_ssdt), >> + "SSDT", table_data->len - nvdimm_ssdt, 1, NULL, "NVDIMM"); >> free_aml_allocator(); > > I prefer ssdt->buf->len to table_data->len - nvdimm_ssdt. > Pls fix by a follow-up patch unless there is a respin. Okay, will do.