DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: announce kernel driver enum removal
@ 2020-08-04 14:07 David Marchand
  2020-08-04 14:29 ` Bruce Richardson
  0 siblings, 1 reply; 4+ messages in thread
From: David Marchand @ 2020-08-04 14:07 UTC (permalink / raw)
  To: dev; +Cc: thomas, grive

rte_kernel_driver is only used by the PCI subsystem and has polluted
ethdev for no reason.
Hide it in the PCI bus driver.

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 doc/guides/rel_notes/deprecation.rst | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 9f73297173..6e86963c39 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -81,6 +81,13 @@ Deprecation Notices
   With this removal, there won't be a need for the mentioned workaround which
   will be reverted.
 
+* pci: The ``rte_kernel_driver`` enum defined in rte_dev.h will be made private
+  to the PCI subsystem as it is used only by the PCI bus driver and PCI
+  drivers.
+  The associated field ``kdrv`` in the ethdev ``rte_eth_dev_data`` structure
+  will be removed as it gave no useful abstracted information to the
+  applications and had no user (neither internal nor external).
+
 * ethdev: Split the ``struct eth_dev_ops`` struct to hide it as much as possible
   will be done in 20.11.
   Currently the ``struct eth_dev_ops`` struct is accessible by the application
-- 
2.23.0


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] doc: announce kernel driver enum removal
  2020-08-04 14:07 [dpdk-dev] [PATCH] doc: announce kernel driver enum removal David Marchand
@ 2020-08-04 14:29 ` Bruce Richardson
  2020-08-04 14:44   ` Gaëtan Rivet
  0 siblings, 1 reply; 4+ messages in thread
From: Bruce Richardson @ 2020-08-04 14:29 UTC (permalink / raw)
  To: David Marchand; +Cc: dev, thomas, grive

On Tue, Aug 04, 2020 at 04:07:52PM +0200, David Marchand wrote:
> rte_kernel_driver is only used by the PCI subsystem and has polluted
> ethdev for no reason.
> Hide it in the PCI bus driver.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
>  doc/guides/rel_notes/deprecation.rst | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 9f73297173..6e86963c39 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -81,6 +81,13 @@ Deprecation Notices
>    With this removal, there won't be a need for the mentioned workaround which
>    will be reverted.
>  
> +* pci: The ``rte_kernel_driver`` enum defined in rte_dev.h will be made private
> +  to the PCI subsystem as it is used only by the PCI bus driver and PCI
> +  drivers.
> +  The associated field ``kdrv`` in the ethdev ``rte_eth_dev_data`` structure
> +  will be removed as it gave no useful abstracted information to the
> +  applications and had no user (neither internal nor external).
> +
>  * ethdev: Split the ``struct eth_dev_ops`` struct to hide it as much as possible
>    will be done in 20.11.
>    Currently the ``struct eth_dev_ops`` struct is accessible by the application
> -- 

Acked-by: Bruce Richardson <bruce.richardson@intel.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] doc: announce kernel driver enum removal
  2020-08-04 14:29 ` Bruce Richardson
@ 2020-08-04 14:44   ` Gaëtan Rivet
  2020-08-05 23:16     ` Thomas Monjalon
  0 siblings, 1 reply; 4+ messages in thread
From: Gaëtan Rivet @ 2020-08-04 14:44 UTC (permalink / raw)
  To: Bruce Richardson; +Cc: David Marchand, dev, thomas

On 04/08/20 15:29 +0100, Bruce Richardson wrote:
> On Tue, Aug 04, 2020 at 04:07:52PM +0200, David Marchand wrote:
> > rte_kernel_driver is only used by the PCI subsystem and has polluted
> > ethdev for no reason.
> > Hide it in the PCI bus driver.
> > 
> > Signed-off-by: David Marchand <david.marchand@redhat.com>
> > ---
> >  doc/guides/rel_notes/deprecation.rst | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> > index 9f73297173..6e86963c39 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -81,6 +81,13 @@ Deprecation Notices
> >    With this removal, there won't be a need for the mentioned workaround which
> >    will be reverted.
> >  
> > +* pci: The ``rte_kernel_driver`` enum defined in rte_dev.h will be made private
> > +  to the PCI subsystem as it is used only by the PCI bus driver and PCI
> > +  drivers.
> > +  The associated field ``kdrv`` in the ethdev ``rte_eth_dev_data`` structure
> > +  will be removed as it gave no useful abstracted information to the
> > +  applications and had no user (neither internal nor external).
> > +
> >  * ethdev: Split the ``struct eth_dev_ops`` struct to hide it as much as possible
> >    will be done in 20.11.
> >    Currently the ``struct eth_dev_ops`` struct is accessible by the application
> > -- 
> 
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Acked-by: Gaetan Rivet <grive@u256.net>

-- 
Gaëtan

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] doc: announce kernel driver enum removal
  2020-08-04 14:44   ` Gaëtan Rivet
@ 2020-08-05 23:16     ` Thomas Monjalon
  0 siblings, 0 replies; 4+ messages in thread
From: Thomas Monjalon @ 2020-08-05 23:16 UTC (permalink / raw)
  To: David Marchand; +Cc: Bruce Richardson, dev, Gaëtan Rivet

04/08/2020 16:44, Gaëtan Rivet:
> On 04/08/20 15:29 +0100, Bruce Richardson wrote:
> > On Tue, Aug 04, 2020 at 04:07:52PM +0200, David Marchand wrote:
> > > rte_kernel_driver is only used by the PCI subsystem and has polluted
> > > ethdev for no reason.
> > > Hide it in the PCI bus driver.
> > > 
> > > Signed-off-by: David Marchand <david.marchand@redhat.com>
> > > ---
> > > --- a/doc/guides/rel_notes/deprecation.rst
> > > +++ b/doc/guides/rel_notes/deprecation.rst
> > > +* pci: The ``rte_kernel_driver`` enum defined in rte_dev.h will be made private
> > > +  to the PCI subsystem as it is used only by the PCI bus driver and PCI
> > > +  drivers.
> > > +  The associated field ``kdrv`` in the ethdev ``rte_eth_dev_data`` structure
> > > +  will be removed as it gave no useful abstracted information to the
> > > +  applications and had no user (neither internal nor external).
> > 
> > Acked-by: Bruce Richardson <bruce.richardson@intel.com>
> Acked-by: Gaetan Rivet <grive@u256.net>
Acked-by: Thomas Monjalon <thomas@monjalon.net>

Applied, thanks



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-08-05 23:16 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-04 14:07 [dpdk-dev] [PATCH] doc: announce kernel driver enum removal David Marchand
2020-08-04 14:29 ` Bruce Richardson
2020-08-04 14:44   ` Gaëtan Rivet
2020-08-05 23:16     ` Thomas Monjalon

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git