From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> To: Boqun Feng <boqun.feng@gmail.com> Cc: linux-pci@vger.kernel.org, linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Michael Kelley <mikelley@microsoft.com>, "K. Y. Srinivasan" <kys@microsoft.com>, Haiyang Zhang <haiyangz@microsoft.com>, Stephen Hemminger <sthemmin@microsoft.com>, Sasha Levin <sashal@kernel.org>, Thomas Gleixner <tglx@linutronix.de>, Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>, "H. Peter Anvin" <hpa@zytor.com>, x86@kernel.org, Andrew Murray <andrew.murray@arm.com>, Bjorn Helgaas <bhelgaas@google.com> Subject: Re: [PATCH v3 0/3] PCI: hv: Generify pci-hyperv.c Date: Mon, 9 Mar 2020 15:03:28 +0000 [thread overview] Message-ID: <20200309150319.GA18350@e121166-lin.cambridge.arm.com> (raw) In-Reply-To: <20200309143548.GB118238@debian-boqun.qqnc3lrjykvubdpftowmye0fmh.lx.internal.cloudapp.net> On Mon, Mar 09, 2020 at 10:35:48PM +0800, Boqun Feng wrote: > Hi Lorenzo, > > On Fri, Feb 21, 2020 at 10:44:54AM +0000, Lorenzo Pieralisi wrote: > > On Fri, Feb 21, 2020 at 10:33:44AM +0800, Boqun Feng wrote: > > > Ping ;-) > > > > > > Any suggestion or plan on this patchset? > > > > Hi, > > > > I shall have a look shortly, thanks. > > > > Any chance you got some time to look into this? I did last week and I applied it now, pci/hv for v5.7. Thanks, Lorenzo > Regards, > Boqun > > > Lorenzo > > > > > Thanks and Regards, > > > Boqun > > > > > > On Mon, Feb 10, 2020 at 11:39:50AM +0800, Boqun Feng wrote: > > > > Hi, > > > > > > > > This is the first part for virtual PCI support of Hyper-V guest on > > > > ARM64. The whole patchset doesn't have any functional change, but only > > > > refactors the pci-hyperv.c code to make it more arch-independent. > > > > > > > > Previous version: > > > > v1: https://lore.kernel.org/lkml/20200121015713.69691-1-boqun.feng@gmail.com/ > > > > v2: https://lore.kernel.org/linux-arm-kernel/20200203050313.69247-1-boqun.feng@gmail.com/ > > > > > > > > Changes since v2: > > > > > > > > * Rebased on 5.6-rc1 > > > > > > > > * Reword commit logs as per Andrew's suggestion. > > > > > > > > * It makes more sense to have a generic interface to set the whole > > > > msi_entry rather than only the "address" field. So change > > > > hv_set_msi_address_from_desc() to hv_set_msi_entry_from_desc(). > > > > Additionally, make it an inline function as per the suggestion > > > > of Andrew and Thomas. > > > > > > > > * Add the missing comment saying the partition_id of > > > > hv_retarget_device_interrupt must be self. > > > > > > > > * Add the explanation for why "__packed" is needed for TLFS > > > > structures. > > > > > > > > I've done compile and boot test of this patchset, also done some tests > > > > with a pass-through NVMe device. > > > > > > > > Suggestions and comments are welcome! > > > > > > > > Regards, > > > > Boqun > > > > > > > > Boqun Feng (3): > > > > PCI: hv: Move hypercall related definitions into tlfs header > > > > PCI: hv: Move retarget related structures into tlfs header > > > > PCI: hv: Introduce hv_msi_entry > > > > > > > > arch/x86/include/asm/hyperv-tlfs.h | 41 +++++++++++++++++++++++++++ > > > > arch/x86/include/asm/mshyperv.h | 8 ++++++ > > > > drivers/pci/controller/pci-hyperv.c | 43 ++--------------------------- > > > > 3 files changed, 52 insertions(+), 40 deletions(-) > > > > > > > > -- > > > > 2.24.1 > > > >
WARNING: multiple messages have this Message-ID (diff)
From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> To: Boqun Feng <boqun.feng@gmail.com> Cc: Sasha Levin <sashal@kernel.org>, linux-hyperv@vger.kernel.org, Stephen Hemminger <sthemmin@microsoft.com>, linux-pci@vger.kernel.org, Haiyang Zhang <haiyangz@microsoft.com>, x86@kernel.org, linux-kernel@vger.kernel.org, Michael Kelley <mikelley@microsoft.com>, Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>, "H. Peter Anvin" <hpa@zytor.com>, Bjorn Helgaas <bhelgaas@google.com>, Andrew Murray <andrew.murray@arm.com>, Thomas Gleixner <tglx@linutronix.de>, "K. Y. Srinivasan" <kys@microsoft.com>, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3 0/3] PCI: hv: Generify pci-hyperv.c Date: Mon, 9 Mar 2020 15:03:28 +0000 [thread overview] Message-ID: <20200309150319.GA18350@e121166-lin.cambridge.arm.com> (raw) In-Reply-To: <20200309143548.GB118238@debian-boqun.qqnc3lrjykvubdpftowmye0fmh.lx.internal.cloudapp.net> On Mon, Mar 09, 2020 at 10:35:48PM +0800, Boqun Feng wrote: > Hi Lorenzo, > > On Fri, Feb 21, 2020 at 10:44:54AM +0000, Lorenzo Pieralisi wrote: > > On Fri, Feb 21, 2020 at 10:33:44AM +0800, Boqun Feng wrote: > > > Ping ;-) > > > > > > Any suggestion or plan on this patchset? > > > > Hi, > > > > I shall have a look shortly, thanks. > > > > Any chance you got some time to look into this? I did last week and I applied it now, pci/hv for v5.7. Thanks, Lorenzo > Regards, > Boqun > > > Lorenzo > > > > > Thanks and Regards, > > > Boqun > > > > > > On Mon, Feb 10, 2020 at 11:39:50AM +0800, Boqun Feng wrote: > > > > Hi, > > > > > > > > This is the first part for virtual PCI support of Hyper-V guest on > > > > ARM64. The whole patchset doesn't have any functional change, but only > > > > refactors the pci-hyperv.c code to make it more arch-independent. > > > > > > > > Previous version: > > > > v1: https://lore.kernel.org/lkml/20200121015713.69691-1-boqun.feng@gmail.com/ > > > > v2: https://lore.kernel.org/linux-arm-kernel/20200203050313.69247-1-boqun.feng@gmail.com/ > > > > > > > > Changes since v2: > > > > > > > > * Rebased on 5.6-rc1 > > > > > > > > * Reword commit logs as per Andrew's suggestion. > > > > > > > > * It makes more sense to have a generic interface to set the whole > > > > msi_entry rather than only the "address" field. So change > > > > hv_set_msi_address_from_desc() to hv_set_msi_entry_from_desc(). > > > > Additionally, make it an inline function as per the suggestion > > > > of Andrew and Thomas. > > > > > > > > * Add the missing comment saying the partition_id of > > > > hv_retarget_device_interrupt must be self. > > > > > > > > * Add the explanation for why "__packed" is needed for TLFS > > > > structures. > > > > > > > > I've done compile and boot test of this patchset, also done some tests > > > > with a pass-through NVMe device. > > > > > > > > Suggestions and comments are welcome! > > > > > > > > Regards, > > > > Boqun > > > > > > > > Boqun Feng (3): > > > > PCI: hv: Move hypercall related definitions into tlfs header > > > > PCI: hv: Move retarget related structures into tlfs header > > > > PCI: hv: Introduce hv_msi_entry > > > > > > > > arch/x86/include/asm/hyperv-tlfs.h | 41 +++++++++++++++++++++++++++ > > > > arch/x86/include/asm/mshyperv.h | 8 ++++++ > > > > drivers/pci/controller/pci-hyperv.c | 43 ++--------------------------- > > > > 3 files changed, 52 insertions(+), 40 deletions(-) > > > > > > > > -- > > > > 2.24.1 > > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-03-09 15:03 UTC|newest] Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-02-10 3:39 [PATCH v3 0/3] PCI: hv: Generify pci-hyperv.c Boqun Feng 2020-02-10 3:39 ` Boqun Feng 2020-02-10 3:39 ` [PATCH v3 1/3] PCI: hv: Move hypercall related definitions into tlfs header Boqun Feng 2020-02-10 3:39 ` Boqun Feng 2020-02-13 4:17 ` Dexuan Cui 2020-02-13 4:17 ` Dexuan Cui 2020-02-10 3:39 ` [PATCH v3 2/3] PCI: hv: Move retarget related structures " Boqun Feng 2020-02-10 3:39 ` Boqun Feng 2020-02-13 4:17 ` Dexuan Cui 2020-02-13 4:17 ` Dexuan Cui 2020-02-13 7:26 ` Boqun Feng 2020-02-13 7:26 ` Boqun Feng 2020-02-13 8:04 ` Dexuan Cui 2020-02-13 8:04 ` Dexuan Cui 2020-02-10 3:39 ` [PATCH v3 3/3] PCI: hv: Introduce hv_msi_entry Boqun Feng 2020-02-10 3:39 ` Boqun Feng 2020-02-13 4:18 ` Dexuan Cui 2020-02-13 4:18 ` Dexuan Cui 2020-02-13 7:14 ` Boqun Feng 2020-02-13 7:14 ` Boqun Feng 2020-02-13 8:05 ` Dexuan Cui 2020-02-13 8:05 ` Dexuan Cui 2020-02-21 2:33 ` [PATCH v3 0/3] PCI: hv: Generify pci-hyperv.c Boqun Feng 2020-02-21 2:33 ` Boqun Feng 2020-02-21 10:44 ` Lorenzo Pieralisi 2020-02-21 10:44 ` Lorenzo Pieralisi 2020-03-09 14:35 ` Boqun Feng 2020-03-09 14:35 ` Boqun Feng 2020-03-09 15:03 ` Lorenzo Pieralisi [this message] 2020-03-09 15:03 ` Lorenzo Pieralisi
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20200309150319.GA18350@e121166-lin.cambridge.arm.com \ --to=lorenzo.pieralisi@arm.com \ --cc=andrew.murray@arm.com \ --cc=bhelgaas@google.com \ --cc=boqun.feng@gmail.com \ --cc=bp@alien8.de \ --cc=haiyangz@microsoft.com \ --cc=hpa@zytor.com \ --cc=kys@microsoft.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-hyperv@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pci@vger.kernel.org \ --cc=mikelley@microsoft.com \ --cc=mingo@redhat.com \ --cc=sashal@kernel.org \ --cc=sthemmin@microsoft.com \ --cc=tglx@linutronix.de \ --cc=x86@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.