From: Thomas Monjalon <thomas.monjalon@6wind.com>
To: Shreyansh Jain <shreyansh.jain@nxp.com>
Cc: dev@dpdk.org, Jan Blunck <jblunck@infradead.org>,
Stephen Hemminger <stephen@networkplumber.org>
Subject: Re: [dpdk-dev] [PATCH] doc: add deprecation note for rework of PCI in EAL
Date: Mon, 13 Feb 2017 15:44:28 +0100 [thread overview]
Message-ID: <2321224.ZvC7RxeBXe@xps13> (raw)
In-Reply-To: <e3c4528d-7be2-e050-4505-17ae5b22a2e2@nxp.com>
2017-02-13 17:30, Shreyansh Jain:
> On Monday 13 February 2017 05:25 PM, Shreyansh Jain wrote:
> > EAL PCI layer is planned to be restructured in 17.05 to unlink it from
> > generic structures like eth_driver, rte_cryptodev_driver, and also move
> > it into a PCI Bus.
> >
> > Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
> > ---
> > doc/guides/rel_notes/deprecation.rst | 12 ++++++++----
> > 1 file changed, 8 insertions(+), 4 deletions(-)
> >
> > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> > index fbe2fcb..b12d435 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -13,10 +13,14 @@ Deprecation Notices
> > has exposed, like the way we have done with uio-pci-generic. This change
> > targets release 17.05.
> >
> > -* ``eth_driver`` is planned to be removed in 17.02. This currently serves as
> > - a placeholder for PMDs to register themselves. Changes for ``rte_bus`` will
> > - provide a way to handle device initialization currently being done in
> > - ``eth_driver``.
>
> Just to highlight, above statement was added by me in 16.11.
> As of now I plan to work on removing rte_pci_driver from eth_driver,
> rather than removing eth_driver all together (which, probably, was
> better idea).
> If someone still wishes to work on its complete removal, we can keep
> the above. (and probably remove the below).
Yes I think we should keep the original idea.
I will work on it with Jan Blunck and Stephen Hemminger I think.
> > +* ABI/API changes are planned for 17.05 for PCI subsystem. This is to
> > + unlink EAL dependency on PCI and to move PCI devices to a PCI specific
> > + bus.
> > +
> > +* ``rte_pci_driver`` is planned to be removed from ``eth_driver`` in 17.05.
> > + This is to unlink the ethernet driver from PCI dependencies.
> > + Similarly, ``rte_pci_driver`` in planned to be removed from
> > + ``rte_cryptodev_driver`` in 17.05.
I am going to reword it in a v2.
next prev parent reply other threads:[~2017-02-13 14:44 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-13 11:55 Shreyansh Jain
2017-02-13 12:00 ` Shreyansh Jain
2017-02-13 14:44 ` Thomas Monjalon [this message]
2017-02-13 16:20 ` [dpdk-dev] [PATCH v2] doc: announce API changes to implement the bus model Thomas Monjalon
2017-02-13 21:53 ` Jan Blunck
2017-02-14 0:09 ` Hemant Agrawal
2017-02-14 0:33 ` Mcnamara, John
2017-02-14 18:57 ` Thomas Monjalon
2017-02-13 21:56 ` [dpdk-dev] [PATCH] doc: add deprecation note for rework of PCI in EAL Jan Blunck
2017-02-14 5:18 ` Shreyansh Jain
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=2321224.ZvC7RxeBXe@xps13 \
--to=thomas.monjalon@6wind.com \
--cc=dev@dpdk.org \
--cc=jblunck@infradead.org \
--cc=shreyansh.jain@nxp.com \
--cc=stephen@networkplumber.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).