DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: announce PCI resources map API removal
@ 2020-08-04  8:47 David Marchand
  2020-08-04  8:52 ` Jerin Jacob
  0 siblings, 1 reply; 4+ messages in thread
From: David Marchand @ 2020-08-04  8:47 UTC (permalink / raw)
  To: dev; +Cc: thomas, dmitry.kozliuk, grive

The PCI resources map API (pci_map_resource/pci_unmap_resource) was
imposing use of Unix mmap flags while it does not make sense on Windows.
This API was only used to internally setup PCI devices in the PCI bus
driver and has no known external users.

Announce its removal in 20.11 with its associated structures.

The workaround implemented in the commit 9d2b24593724 ("pci: keep API
compatibility with mmap values") will be removed at the same time.

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

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index ea4cfa7a48..24808c002c 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -67,6 +67,17 @@ Deprecation Notices
   us extending existing enum/define.
   One solution can be using a fixed size array instead of ``.*MAX.*`` value.
 
+* pci: The PCI resources map API (``pci_map_resource`` and
+  ``pci_unmap_resource``) was not abstracting the Unix mmap flags (see the
+  workaround for Windows support implemented in the commit
+  9d2b24593724 ("pci: keep API compatibility with mmap values")).
+  This API will be removed from the public API in 20.11 and moved to the PCI
+  bus driver along with the PCI resources lists and associated structures
+  (``pci_map``, ``pci_msix_table``, ``mapped_pci_resource`` and
+  ``mapped_pci_res_list``).
+  With this removal, there won't be a need for the mentioned workaround which
+  will be reverted.
+
 * 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 PCI resources map API removal
  2020-08-04  8:47 [dpdk-dev] [PATCH] doc: announce PCI resources map API removal David Marchand
@ 2020-08-04  8:52 ` Jerin Jacob
  2020-08-04 11:20   ` Gaëtan Rivet
  0 siblings, 1 reply; 4+ messages in thread
From: Jerin Jacob @ 2020-08-04  8:52 UTC (permalink / raw)
  To: David Marchand; +Cc: dpdk-dev, Thomas Monjalon, Dmitry Kozlyuk, Gaetan Rivet

On Tue, Aug 4, 2020 at 2:18 PM David Marchand <david.marchand@redhat.com> wrote:
>
> The PCI resources map API (pci_map_resource/pci_unmap_resource) was
> imposing use of Unix mmap flags while it does not make sense on Windows.
> This API was only used to internally setup PCI devices in the PCI bus
> driver and has no known external users.
>
> Announce its removal in 20.11 with its associated structures.
>
> The workaround implemented in the commit 9d2b24593724 ("pci: keep API
> compatibility with mmap values") will be removed at the same time.
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>

Acked-by: Jerin Jacob <jerinj@marvell.com>


> ---
>  doc/guides/rel_notes/deprecation.rst | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index ea4cfa7a48..24808c002c 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -67,6 +67,17 @@ Deprecation Notices
>    us extending existing enum/define.
>    One solution can be using a fixed size array instead of ``.*MAX.*`` value.
>
> +* pci: The PCI resources map API (``pci_map_resource`` and
> +  ``pci_unmap_resource``) was not abstracting the Unix mmap flags (see the
> +  workaround for Windows support implemented in the commit
> +  9d2b24593724 ("pci: keep API compatibility with mmap values")).
> +  This API will be removed from the public API in 20.11 and moved to the PCI
> +  bus driver along with the PCI resources lists and associated structures
> +  (``pci_map``, ``pci_msix_table``, ``mapped_pci_resource`` and
> +  ``mapped_pci_res_list``).
> +  With this removal, there won't be a need for the mentioned workaround which
> +  will be reverted.
> +
>  * 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 PCI resources map API removal
  2020-08-04  8:52 ` Jerin Jacob
@ 2020-08-04 11:20   ` Gaëtan Rivet
  2020-08-05 23:21     ` Thomas Monjalon
  0 siblings, 1 reply; 4+ messages in thread
From: Gaëtan Rivet @ 2020-08-04 11:20 UTC (permalink / raw)
  To: Jerin Jacob; +Cc: David Marchand, dpdk-dev, Thomas Monjalon, Dmitry Kozlyuk

On 04/08/20 14:22 +0530, Jerin Jacob wrote:
> On Tue, Aug 4, 2020 at 2:18 PM David Marchand <david.marchand@redhat.com> wrote:
> >
> > The PCI resources map API (pci_map_resource/pci_unmap_resource) was
> > imposing use of Unix mmap flags while it does not make sense on Windows.
> > This API was only used to internally setup PCI devices in the PCI bus
> > driver and has no known external users.
> >
> > Announce its removal in 20.11 with its associated structures.
> >
> > The workaround implemented in the commit 9d2b24593724 ("pci: keep API
> > compatibility with mmap values") will be removed at the same time.
> >
> > Signed-off-by: David Marchand <david.marchand@redhat.com>
> 
> Acked-by: Jerin Jacob <jerinj@marvell.com>
> 

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

> 
> > ---
> >  doc/guides/rel_notes/deprecation.rst | 11 +++++++++++
> >  1 file changed, 11 insertions(+)
> >
> > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> > index ea4cfa7a48..24808c002c 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -67,6 +67,17 @@ Deprecation Notices
> >    us extending existing enum/define.
> >    One solution can be using a fixed size array instead of ``.*MAX.*`` value.
> >
> > +* pci: The PCI resources map API (``pci_map_resource`` and
> > +  ``pci_unmap_resource``) was not abstracting the Unix mmap flags (see the
> > +  workaround for Windows support implemented in the commit
> > +  9d2b24593724 ("pci: keep API compatibility with mmap values")).
> > +  This API will be removed from the public API in 20.11 and moved to the PCI
> > +  bus driver along with the PCI resources lists and associated structures
> > +  (``pci_map``, ``pci_msix_table``, ``mapped_pci_resource`` and
> > +  ``mapped_pci_res_list``).
> > +  With this removal, there won't be a need for the mentioned workaround which
> > +  will be reverted.
> > +
> >  * 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
> >

-- 
Gaëtan

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

* Re: [dpdk-dev] [PATCH] doc: announce PCI resources map API removal
  2020-08-04 11:20   ` Gaëtan Rivet
@ 2020-08-05 23:21     ` Thomas Monjalon
  0 siblings, 0 replies; 4+ messages in thread
From: Thomas Monjalon @ 2020-08-05 23:21 UTC (permalink / raw)
  To: David Marchand; +Cc: Jerin Jacob, dev, Dmitry Kozlyuk, Gaëtan Rivet

04/08/2020 13:20, Gaëtan Rivet:
> On 04/08/20 14:22 +0530, Jerin Jacob wrote:
> > On Tue, Aug 4, 2020 at 2:18 PM David Marchand <david.marchand@redhat.com> wrote:
> > >
> > > The PCI resources map API (pci_map_resource/pci_unmap_resource) was
> > > imposing use of Unix mmap flags while it does not make sense on Windows.
> > > This API was only used to internally setup PCI devices in the PCI bus
> > > driver and has no known external users.
> > >
> > > Announce its removal in 20.11 with its associated structures.
> > >
> > > The workaround implemented in the commit 9d2b24593724 ("pci: keep API
> > > compatibility with mmap values") will be removed at the same time.
> > >
> > > Signed-off-by: David Marchand <david.marchand@redhat.com>
> > Acked-by: Jerin Jacob <jerinj@marvell.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:21 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-04  8:47 [dpdk-dev] [PATCH] doc: announce PCI resources map API removal David Marchand
2020-08-04  8:52 ` Jerin Jacob
2020-08-04 11:20   ` Gaëtan Rivet
2020-08-05 23:21     ` 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