All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@lst.de>
To: Max Gurtovoy <maxg@mellanox.com>
Cc: "'gregkh@linuxfoundation.org'" <gregkh@linuxfoundation.org>,
	Shlomi Nimrodi <shlomin@mellanox.com>,
	Paul Mackerras <paulus@samba.org>,
	Frederic Barrat <fbarrat@linux.ibm.com>,
	Oren Duer <oren@mellanox.com>, Idan Werpoler <Idanw@mellanox.com>,
	"linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
	Christoph Hellwig <hch@lst.de>
Subject: Re: [PATCH 1/3] powerpc/powernv: remove the unused pnv_pci_set_p2p function
Date: Tue, 9 Jul 2019 15:59:35 +0200	[thread overview]
Message-ID: <20190709135935.GA2511@lst.de> (raw)
In-Reply-To: <AM0PR05MB5810FF3B4A45B3CA722489EEB6F10@AM0PR05MB5810.eurprd05.prod.outlook.com>

On Tue, Jul 09, 2019 at 01:49:04PM +0000, Max Gurtovoy wrote:
> Hi Greg/Christoph,
> Can we leave it meanwhile till we'll find a general solution (for the upcoming kernel) ?
> I guess we can somehow generalize the P2P initialization process for PPC and leave it empty for now for other archs.
> Or maybe we can find some other solution (sysfs/configfs/module param), but it will take time since we'll need to work closely with the IBM pci guys that wrote this code.

We do not keep code without in-tree users around, especially not if
we have a better API with in-tree users.

AFAICS the only thing you'll need is to wire up the enable/disable
calls.

  reply	other threads:[~2019-07-09 14:03 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-26 12:49 remove dead powernv code Christoph Hellwig
2019-04-26 12:49 ` [PATCH 1/3] powerpc/powernv: remove the unused pnv_pci_set_p2p function Christoph Hellwig
2019-05-06  8:46   ` Frederic Barrat
2019-05-06  9:02     ` Christoph Hellwig
2019-05-23  7:52     ` Christoph Hellwig
2019-07-09 13:49       ` Max Gurtovoy
2019-07-09 13:59         ` Christoph Hellwig [this message]
2019-07-09 14:31           ` Max Gurtovoy
2019-07-09 14:32             ` Christoph Hellwig
2019-07-09 14:37               ` Max Gurtovoy
2019-07-09 14:40                 ` Christoph Hellwig
2019-07-09 15:06                   ` Max Gurtovoy
2019-07-09 15:08                     ` Christoph Hellwig
2019-07-09 20:02                     ` 'gregkh@linuxfoundation.org'
2019-04-26 12:49 ` [PATCH 2/3] powerpc/powernv: remove pnv_pci_{enable,disable}_tunnel Christoph Hellwig
2019-04-26 12:49 ` [PATCH 3/3] powerpc/powernv: remove dead NPU DMA code 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=20190709135935.GA2511@lst.de \
    --to=hch@lst.de \
    --cc=Idanw@mellanox.com \
    --cc=fbarrat@linux.ibm.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=maxg@mellanox.com \
    --cc=oren@mellanox.com \
    --cc=paulus@samba.org \
    --cc=shlomin@mellanox.com \
    /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.