From mboxrd@z Thu Jan  1 00:00:00 1970
From: Xiao Guangrong <guangrong.xiao@linux.intel.com>
Subject: Re: [PATCH v4 28/33] nvdimm acpi: support
 DSM_FUN_IMPLEMENTED function
Date: Wed, 21 Oct 2015 00:29:36 +0800
Message-ID: <56266BF0.3010905@linux.intel.com>
References: <1445216059-88521-1-git-send-email-guangrong.xiao@linux.intel.com>
	<1445216059-88521-29-git-send-email-guangrong.xiao@linux.intel.com>
	<20151020155149.GB2772@stefanha-thinkpad.home>
	<56266B3B.4080407@linux.intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Cc: ehabkost@redhat.com, kvm@vger.kernel.org, mst@redhat.com, gleb@kernel.org,
	mtosatti@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com,
	imammedo@redhat.com, pbonzini@redhat.com,
	dan.j.williams@intel.com, rth@twiddle.net
To: Stefan Hajnoczi <stefanha@gmail.com>
Return-path: <qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org>
In-Reply-To: <56266B3B.4080407@linux.intel.com>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org
Sender: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org
List-Id: kvm.vger.kernel.org



On 10/21/2015 12:26 AM, Xiao Guangrong wrote:
>
>
> On 10/20/2015 11:51 PM, Stefan Hajnoczi wrote:
>> On Mon, Oct 19, 2015 at 08:54:14AM +0800, Xiao Guangrong wrote:
>>> +exit:
>>> +    /* Write our output result to dsm memory. */
>>> +    ((dsm_out *)dsm_ram_addr)->len = out->len;
>>
>> Missing byteswap?
>>
>> I thought you were going to remove this field because it wasn't needed
>> by the guest.
>>
>
> The @len is the size of _DSM result buffer, for example, for the function of
> DSM_FUN_IMPLEMENTED the result buffer is 8 bytes, and for
> DSM_DEV_FUN_NAMESPACE_LABEL_SIZE the buffer size is 4 bytes. It tells ASL code

Sorry, s/DSM_DEV_FUN_NAMESPACE_LABEL_SIZE/DSM_DEV_FUN_SET_NAMESPACE_LABEL_DATA

> how much size of memory we need to return to the _DSM caller.
>
> In _DSM code, it's handled like this:
>
> "RLEN" is @len, “OBUF” is the left memory in DSM page.
>
>          /* get @len*/
>          aml_append(method, aml_store(aml_name("RLEN"), aml_local(6)));
>          /* @len << 3 to get bits. */
>          aml_append(method, aml_store(aml_shiftleft(aml_local(6),
>                                         aml_int(3)), aml_local(6)));
>
>          /* get @len << 3 bits from OBUF, and return it to the caller. */
>          aml_append(method, aml_create_field(aml_name("ODAT"), aml_int(0),
>                                              aml_local(6) , "OBUF"));
>
> Since @len is our internally used, it's not return to guest, so i did not do
> byteswap here.