From: Christoph Hellwig <hch@lst.de>
To: "Cédric Le Goater" <clg@kaod.org>
Cc: Alexey Kardashevskiy <aik@ozlabs.ru>,
linux-kernel@vger.kernel.org, Paul Mackerras <paulus@samba.org>,
Frederic Barrat <fbarrat@linux.ibm.com>,
Oliver O'Halloran <oohall@gmail.com>,
linuxppc-dev@lists.ozlabs.org, Christoph Hellwig <hch@lst.de>
Subject: Re: [PATCH 1/4] powerpc/powernv: remove the unused pnv_pci_set_p2p function
Date: Tue, 25 Jun 2019 10:28:40 +0200 [thread overview]
Message-ID: <20190625082840.GA31969@lst.de> (raw)
In-Reply-To: <113fb518-0f5a-8ced-8391-abe48869a0cb@kaod.org>
On Tue, Jun 25, 2019 at 10:26:03AM +0200, Cédric Le Goater wrote:
> > @@ -280,13 +280,6 @@ int64_t opal_xive_allocate_irq(uint32_t chip_id);
> > int64_t opal_xive_free_irq(uint32_t girq);
> > int64_t opal_xive_sync(uint32_t type, uint32_t id);
> > int64_t opal_xive_dump(uint32_t type, uint32_t id);
> > -int64_t opal_xive_get_queue_state(uint64_t vp, uint32_t prio,
> > - __be32 *out_qtoggle,
> > - __be32 *out_qindex);
> > -int64_t opal_xive_set_queue_state(uint64_t vp, uint32_t prio,
> > - uint32_t qtoggle,
> > - uint32_t qindex);
> > -int64_t opal_xive_get_vp_state(uint64_t vp, __be64 *out_w01);
>
>
> This hunk seems unrelated.
>
> These OPAL calls are new. They are used by the XIVE KVM device
> to get/set the interrupt controller state of a guest.
>
>
> > int64_t opal_pci_set_p2p(uint64_t phb_init, uint64_t phb_target,
> > uint64_t desc, uint16_t pe_number);
>
> I suppose this is the one ^ you wanted to remove.
Thanks. I'm pretty sure I had this fixed up before due to a builtbot
warning, but somehow the old version popped up again.
WARNING: multiple messages have this Message-ID (diff)
From: Christoph Hellwig <hch@lst.de>
To: "Cédric Le Goater" <clg@kaod.org>
Cc: Christoph Hellwig <hch@lst.de>,
Benjamin Herrenschmidt <benh@kernel.crashing.org>,
Paul Mackerras <paulus@samba.org>,
Michael Ellerman <mpe@ellerman.id.au>,
Alexey Kardashevskiy <aik@ozlabs.ru>,
Frederic Barrat <fbarrat@linux.ibm.com>,
Oliver O'Halloran <oohall@gmail.com>,
linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/4] powerpc/powernv: remove the unused pnv_pci_set_p2p function
Date: Tue, 25 Jun 2019 10:28:40 +0200 [thread overview]
Message-ID: <20190625082840.GA31969@lst.de> (raw)
In-Reply-To: <113fb518-0f5a-8ced-8391-abe48869a0cb@kaod.org>
On Tue, Jun 25, 2019 at 10:26:03AM +0200, Cédric Le Goater wrote:
> > @@ -280,13 +280,6 @@ int64_t opal_xive_allocate_irq(uint32_t chip_id);
> > int64_t opal_xive_free_irq(uint32_t girq);
> > int64_t opal_xive_sync(uint32_t type, uint32_t id);
> > int64_t opal_xive_dump(uint32_t type, uint32_t id);
> > -int64_t opal_xive_get_queue_state(uint64_t vp, uint32_t prio,
> > - __be32 *out_qtoggle,
> > - __be32 *out_qindex);
> > -int64_t opal_xive_set_queue_state(uint64_t vp, uint32_t prio,
> > - uint32_t qtoggle,
> > - uint32_t qindex);
> > -int64_t opal_xive_get_vp_state(uint64_t vp, __be64 *out_w01);
>
>
> This hunk seems unrelated.
>
> These OPAL calls are new. They are used by the XIVE KVM device
> to get/set the interrupt controller state of a guest.
>
>
> > int64_t opal_pci_set_p2p(uint64_t phb_init, uint64_t phb_target,
> > uint64_t desc, uint16_t pe_number);
>
> I suppose this is the one ^ you wanted to remove.
Thanks. I'm pretty sure I had this fixed up before due to a builtbot
warning, but somehow the old version popped up again.
next prev parent reply other threads:[~2019-06-25 8:31 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-25 8:15 remove unused powernv code v2 Christoph Hellwig
2019-06-25 8:15 ` Christoph Hellwig
2019-06-25 8:15 ` [PATCH 1/4] powerpc/powernv: remove the unused pnv_pci_set_p2p function Christoph Hellwig
2019-06-25 8:15 ` Christoph Hellwig
2019-06-25 8:26 ` Cédric Le Goater
2019-06-25 8:28 ` Christoph Hellwig [this message]
2019-06-25 8:28 ` Christoph Hellwig
2019-06-25 8:15 ` [PATCH 2/4] powerpc/powernv: remove the unused tunneling exports Christoph Hellwig
2019-06-25 8:15 ` Christoph Hellwig
2019-06-25 8:15 ` [PATCH 3/4] powerpc/powernv: remove unused NPU DMA code Christoph Hellwig
2019-06-25 8:15 ` Christoph Hellwig
2019-06-25 8:15 ` [PATCH 4/4] powerpc/powernv: remove the unused vas_win_paste_addr and vas_win_id functions Christoph Hellwig
2019-06-25 8:15 ` Christoph Hellwig
-- strict thread matches above, loose matches on Subject: below --
2019-06-25 14:52 remove unused powernv code v3 Christoph Hellwig
2019-06-25 14:52 ` [PATCH 1/4] powerpc/powernv: remove the unused pnv_pci_set_p2p function Christoph Hellwig
2019-06-25 14:52 ` Christoph Hellwig
2019-07-03 14:27 ` Michael Ellerman
2019-07-03 14:33 ` Christoph Hellwig
2019-07-03 14:33 ` Christoph Hellwig
2019-05-23 7:49 remove dead powernv code v2 Christoph Hellwig
2019-05-23 7:49 ` [PATCH 1/4] powerpc/powernv: remove the unused pnv_pci_set_p2p function Christoph Hellwig
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=20190625082840.GA31969@lst.de \
--to=hch@lst.de \
--cc=aik@ozlabs.ru \
--cc=clg@kaod.org \
--cc=fbarrat@linux.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=oohall@gmail.com \
--cc=paulus@samba.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: link
Be 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.