From mboxrd@z Thu Jan 1 00:00:00 1970 Received: by 2002:a05:6512:3aa:0:0:0:0 with SMTP id v10csp410707lfp; Wed, 29 Jan 2020 02:44:34 -0800 (PST) X-Google-Smtp-Source: APXvYqwpcE2lVPNYz1pbl6c9RDGWI0RnzT0FhFunEFhKPXeWweFifvEvZ2pmp39DSLnsI+fmRDqv X-Received: by 2002:ac8:7501:: with SMTP id u1mr26478255qtq.149.1580294669047; Wed, 29 Jan 2020 02:44:29 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1580294668; cv=none; d=google.com; s=arc-20160816; b=VIhVPjErrLsR4NxleQUELRSQxOX9cPQlzPbm3cmW5jQbMy16XBeQ+kW2Sc40en7XGt 93jj2OAxxgYJgClf1SIOISVmURuptIpmJvLPfbHDclvclOATXerbRvcahnX8wxPrJJvi I0lSpu7YhAqocQfufQidPNnpyGe6olRqoKXt0CKWOEacLY3G/O/u7XOv+78H1wTk5H9j 48pd8bhWtPLHS/IlVMu4Pya1DZ/++l977tIshpRIw9EhJx/1okcjK4Uvg6az65f3wpBK aNwOD781EBcU9BnDUiqQbaDqDbi31R+5qlEJYXfBFAXdTfvYDOY99hVX+vJkZoUF08gh P6JQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:cc:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:mime-version :content-transfer-encoding:content-language:accept-language :in-reply-to:references:message-id:date:thread-index:thread-topic :subject:to:from; bh=TxY7eHwDd8ZRfbOxnmyWFcvuVNz/IQ2R71CNy8TGYeA=; b=Dn1uqP/UzX6JEFcmnkJZSxmGzM2N2IL54U0xczuTBrrDsQo3B92VZ3HvIiF65Vd80H ozIg5hJBHGYi6rVQ03aIWpj0Ruh5aBuBtmSuDsHY/b+yV30P5dnmzpfzpSRhlf68LQQw ZZE51t+ULbSznsV+R5WjKAeEo9F1nsLYwwqS1w+jFrCVuEUHLVRoTIbDtjxYcaV1T1cc pFLho2AGJu2P/M9H6YkiocU0CgifezCf12mEi1r0idi4J1s2eI0votg97+Z2k/xEb+HL AKrVVpVUr2afRBr2LswBZlklRuDbgVy8CJaK9o82aGv9OnnDcy4Spv1mJ4dhVepxyXEb jRSg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of qemu-arm-bounces+alex.bennee=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-arm-bounces+alex.bennee=linaro.org@nongnu.org" Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id l19si1365766qvt.203.2020.01.29.02.44.28 for (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Wed, 29 Jan 2020 02:44:28 -0800 (PST) Received-SPF: pass (google.com: domain of qemu-arm-bounces+alex.bennee=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; spf=pass (google.com: domain of qemu-arm-bounces+alex.bennee=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-arm-bounces+alex.bennee=linaro.org@nongnu.org" Received: from localhost ([::1]:44108 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iwkpg-0005wS-Bi for alex.bennee@linaro.org; Wed, 29 Jan 2020 05:44:28 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:49342) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iwkpY-0005sJ-DX for qemu-arm@nongnu.org; Wed, 29 Jan 2020 05:44:22 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iwkpX-0002OE-13 for qemu-arm@nongnu.org; Wed, 29 Jan 2020 05:44:20 -0500 Received: from lhrrgout.huawei.com ([185.176.76.210]:2067 helo=huawei.com) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1iwkpW-0002O3-QP; Wed, 29 Jan 2020 05:44:18 -0500 Received: from lhreml707-cah.china.huawei.com (unknown [172.18.7.108]) by Forcepoint Email with ESMTP id 075591C4EC060FD8C653; Wed, 29 Jan 2020 10:44:17 +0000 (GMT) Received: from lhreml708-chm.china.huawei.com (10.201.108.57) by lhreml707-cah.china.huawei.com (10.201.108.48) with Microsoft SMTP Server (TLS) id 14.3.408.0; Wed, 29 Jan 2020 10:44:16 +0000 Received: from lhreml710-chm.china.huawei.com (10.201.108.61) by lhreml708-chm.china.huawei.com (10.201.108.57) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1713.5; Wed, 29 Jan 2020 10:44:16 +0000 Received: from lhreml710-chm.china.huawei.com ([169.254.81.184]) by lhreml710-chm.china.huawei.com ([169.254.81.184]) with mapi id 15.01.1713.004; Wed, 29 Jan 2020 10:44:16 +0000 From: Shameerali Kolothum Thodi To: Auger Eric , "qemu-devel@nongnu.org" , "qemu-arm@nongnu.org" , "imammedo@redhat.com" Subject: RE: [PATCH v2 0/7] ARM virt: Add NVDIMM support Thread-Topic: [PATCH v2 0/7] ARM virt: Add NVDIMM support Thread-Index: AQHVzV42yTrHPe4JeEKQykfFHPYCE6gARKaAgAFB8XA= Date: Wed, 29 Jan 2020 10:44:16 +0000 Message-ID: References: <20200117174522.22044-1-shameerali.kolothum.thodi@huawei.com> <7b338b7c-fd47-930b-53d5-894711c69941@redhat.com> In-Reply-To: <7b338b7c-fd47-930b-53d5-894711c69941@redhat.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.202.227.237] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-CFilter-Loop: Reflected X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 185.176.76.210 X-BeenThere: qemu-arm@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "peter.maydell@linaro.org" , "xiaoguangrong.eric@gmail.com" , "mst@redhat.com" , Linuxarm , "shannon.zhaosl@gmail.com" , "xuwei \(O\)" , "lersek@redhat.com" Errors-To: qemu-arm-bounces+alex.bennee=linaro.org@nongnu.org Sender: "Qemu-arm" X-TUID: 8vObfg0FZpkC Hi Eric, > -----Original Message----- > From: Auger Eric [mailto:eric.auger@redhat.com] > Sent: 28 January 2020 15:29 > To: Shameerali Kolothum Thodi ; > qemu-devel@nongnu.org; qemu-arm@nongnu.org; imammedo@redhat.com > Cc: peter.maydell@linaro.org; shannon.zhaosl@gmail.com; mst@redhat.com; > xiaoguangrong.eric@gmail.com; xuwei (O) ; > lersek@redhat.com; Linuxarm > Subject: Re: [PATCH v2 0/7] ARM virt: Add NVDIMM support >=20 > Hi Shameer, >=20 > On 1/17/20 6:45 PM, Shameer Kolothum wrote: > > This series adds NVDIMM support to arm/virt platform. > > The series reuses some of the patches posted by Eric > > in his earlier attempt here[1]. > > > > Patch #1 is a fix to the Guest reboot issue on NVDIMM > > hot add case described here[2] and patch #2 is another > > fix to the nvdimm aml issue discussed here[3]. > > > > I have done a basic sanity testing of NVDIMM deviecs > > with Guest booting with both ACPI and DT. Further testing > > is always welcome. > > > > Please let me know your feedback. >=20 >=20 > With this version, I do not get the former spurious warning reported on v= 1. >=20 > I can see the nvdimm device topology using ndctl. So it looks fine to me. Thanks for giving it a spin and confirming.=20 > Unfortunately we cannot test with DAX as kernel dependencies are not yet > resolved yet but this is an independent problem. True. I did previously test DAX with "arm64/mm: Enable memory hot remove" Patch series and that seems to work fine. Cheers, Shameer =20 > Thanks >=20 > Eric > > > > Thanks, > > Shameer > > > > [1] https://patchwork.kernel.org/cover/10830777/ > > [2] https://patchwork.kernel.org/patch/11154757/ > > [3] https://patchwork.kernel.org/cover/11174959/ > > > > v1 --> v2 > > -Reworked patch #1 and now fix is inside qemu_ram_resize(). > > -Added patch #2 to fix the nvdim aml issue. > > -Dropped support to DT cold plug. > > -Updated test_acpi_virt_tcg_memhp() with pc-dimm and nvdimms(patch > #7) > > > > Kwangwoo Lee (2): > > nvdimm: Use configurable ACPI IO base and size > > hw/arm/virt: Add nvdimm hot-plug infrastructure > > > > Shameer Kolothum (5): > > exec: Fix for qemu_ram_resize() callback > > hw/acpi/nvdimm: Fix for NVDIMM incorrect DSM output buffer length > > hw/arm/virt: Add nvdimm hotplug support > > tests: Update ACPI tables list for upcoming arm/virt test changes > > tests/bios-tables-test: Update arm/virt memhp test > > > > docs/specs/acpi_hw_reduced_hotplug.rst | 1 + > > exec.c | 36 +++++++---- > > hw/acpi/generic_event_device.c | 13 ++++ > > hw/acpi/nvdimm.c | 68 > +++++++++++++++++---- > > hw/arm/Kconfig | 1 + > > hw/arm/virt-acpi-build.c | 6 ++ > > hw/arm/virt.c | 35 +++++++++-- > > hw/i386/acpi-build.c | 6 ++ > > hw/i386/acpi-build.h | 3 + > > hw/i386/pc_piix.c | 2 + > > hw/i386/pc_q35.c | 2 + > > hw/mem/Kconfig | 2 +- > > include/exec/ram_addr.h | 5 +- > > include/hw/acpi/generic_event_device.h | 1 + > > include/hw/arm/virt.h | 1 + > > include/hw/mem/nvdimm.h | 3 + > > tests/data/acpi/virt/NFIT.memhp | 0 > > tests/data/acpi/virt/SSDT.memhp | 0 > > tests/qtest/bios-tables-test-allowed-diff.h | 5 ++ > > tests/qtest/bios-tables-test.c | 9 ++- > > 20 files changed, 163 insertions(+), 36 deletions(-) > > create mode 100644 tests/data/acpi/virt/NFIT.memhp > > create mode 100644 tests/data/acpi/virt/SSDT.memhp > >