DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] doc: announce marking bus object as internal
@ 2022-06-30  9:41 David Marchand
  2022-06-30  9:54 ` Bruce Richardson
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: David Marchand @ 2022-06-30  9:41 UTC (permalink / raw)
  To: dev, techboard; +Cc: Ray Kinsella

rte_bus is unnecessarily exposed in the public API/ABI.
Besides, we had cases where extending rte_bus was necessary.
Announce that rte_bus will be made opaque in the public API and mark
associated API as internal.

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
A RFC series of the intended changes is available at:
https://patches.dpdk.org/project/dpdk/list/?series=23811&state=%2A&archive=both

---
 doc/guides/rel_notes/deprecation.rst | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 64d649777a..a9fd6676be 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -32,6 +32,12 @@ Deprecation Notices
   ``__atomic_thread_fence`` must be used for patches that need to be merged in
   20.08 onwards. This change will not introduce any performance degradation.
 
+* bus: The ``rte_bus`` object will be made opaque in DPDK 22.11.
+  The goal is to remove it from the public ABI and make this object extendable.
+  As a side effect, registering a bus will be marked as an internal API:
+  external users may still register their bus using a new driver header (see
+  ``enable_driver_sdk`` meson option).
+
 * mempool: Helper macro ``MEMPOOL_HEADER_SIZE()`` is deprecated and will
   be removed in DPDK 22.11. The replacement macro
   ``RTE_MEMPOOL_HEADER_SIZE()`` is internal only.
-- 
2.36.1


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

* Re: [PATCH] doc: announce marking bus object as internal
  2022-06-30  9:41 [PATCH] doc: announce marking bus object as internal David Marchand
@ 2022-06-30  9:54 ` Bruce Richardson
  2022-06-30 14:36   ` Thomas Monjalon
  2022-07-04 14:29 ` Kevin Laatz
  2022-07-15 16:16 ` Thomas Monjalon
  2 siblings, 1 reply; 7+ messages in thread
From: Bruce Richardson @ 2022-06-30  9:54 UTC (permalink / raw)
  To: David Marchand; +Cc: dev, techboard, Ray Kinsella

On Thu, Jun 30, 2022 at 11:41:53AM +0200, David Marchand wrote:
> rte_bus is unnecessarily exposed in the public API/ABI.
> Besides, we had cases where extending rte_bus was necessary.
> Announce that rte_bus will be made opaque in the public API and mark
> associated API as internal.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
> A RFC series of the intended changes is available at:
> https://patches.dpdk.org/project/dpdk/list/?series=23811&state=%2A&archive=both
> 
> ---
Acked-by: Bruce Richardson <bruce.richardson@intel.com>

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

* Re: [PATCH] doc: announce marking bus object as internal
  2022-06-30  9:54 ` Bruce Richardson
@ 2022-06-30 14:36   ` Thomas Monjalon
  0 siblings, 0 replies; 7+ messages in thread
From: Thomas Monjalon @ 2022-06-30 14:36 UTC (permalink / raw)
  To: David Marchand; +Cc: techboard, dev, Ray Kinsella, Bruce Richardson

30/06/2022 11:54, Bruce Richardson:
> On Thu, Jun 30, 2022 at 11:41:53AM +0200, David Marchand wrote:
> > rte_bus is unnecessarily exposed in the public API/ABI.
> > Besides, we had cases where extending rte_bus was necessary.
> > Announce that rte_bus will be made opaque in the public API and mark
> > associated API as internal.
> > 
> > Signed-off-by: David Marchand <david.marchand@redhat.com>
> > ---
> > A RFC series of the intended changes is available at:
> > https://patches.dpdk.org/project/dpdk/list/?series=23811&state=%2A&archive=both
> > 
> > ---
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Acked-by: Thomas Monjalon <thomas@monjalon.net>



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

* Re: [PATCH] doc: announce marking bus object as internal
  2022-06-30  9:41 [PATCH] doc: announce marking bus object as internal David Marchand
  2022-06-30  9:54 ` Bruce Richardson
@ 2022-07-04 14:29 ` Kevin Laatz
  2022-07-05  4:43   ` Hemant Agrawal
  2022-07-15 16:16 ` Thomas Monjalon
  2 siblings, 1 reply; 7+ messages in thread
From: Kevin Laatz @ 2022-07-04 14:29 UTC (permalink / raw)
  To: David Marchand, dev, techboard; +Cc: Ray Kinsella

On 30/06/2022 10:41, David Marchand wrote:
> rte_bus is unnecessarily exposed in the public API/ABI.
> Besides, we had cases where extending rte_bus was necessary.
> Announce that rte_bus will be made opaque in the public API and mark
> associated API as internal.
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
> A RFC series of the intended changes is available at:
> https://patches.dpdk.org/project/dpdk/list/?series=23811&state=%2A&archive=both
>
> ---
>   doc/guides/rel_notes/deprecation.rst | 6 ++++++
>   1 file changed, 6 insertions(+)
>
Acked-by: Kevin Laatz <kevin.laatz@intel.com>


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

* Re: [PATCH] doc: announce marking bus object as internal
  2022-07-04 14:29 ` Kevin Laatz
@ 2022-07-05  4:43   ` Hemant Agrawal
  2022-07-05  6:59     ` Andrew Rybchenko
  0 siblings, 1 reply; 7+ messages in thread
From: Hemant Agrawal @ 2022-07-05  4:43 UTC (permalink / raw)
  To: Kevin Laatz, David Marchand, dev, techboard; +Cc: Ray Kinsella

Acked-by:  Hemant Agrawal <hemant.agrawal@nxp.com>

On 7/4/2022 7:59 PM, Kevin Laatz wrote:
> On 30/06/2022 10:41, David Marchand wrote:
>> rte_bus is unnecessarily exposed in the public API/ABI.
>> Besides, we had cases where extending rte_bus was necessary.
>> Announce that rte_bus will be made opaque in the public API and mark
>> associated API as internal.
>>
>> Signed-off-by: David Marchand <david.marchand@redhat.com>
>> ---
>> A RFC series of the intended changes is available at:
>> https://patches.dpdk.org/project/dpdk/list/?series=23811&state=%2A&archive=both 
>>
>>
>> ---
>>   doc/guides/rel_notes/deprecation.rst | 6 ++++++
>>   1 file changed, 6 insertions(+)
>>
> Acked-by: Kevin Laatz <kevin.laatz@intel.com>
>

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

* Re: [PATCH] doc: announce marking bus object as internal
  2022-07-05  4:43   ` Hemant Agrawal
@ 2022-07-05  6:59     ` Andrew Rybchenko
  0 siblings, 0 replies; 7+ messages in thread
From: Andrew Rybchenko @ 2022-07-05  6:59 UTC (permalink / raw)
  To: hemant.agrawal, Kevin Laatz, David Marchand, dev, techboard; +Cc: Ray Kinsella

On 7/5/22 07:43, Hemant Agrawal wrote:
> Acked-by:  Hemant Agrawal <hemant.agrawal@nxp.com>
> 
> On 7/4/2022 7:59 PM, Kevin Laatz wrote:
>> On 30/06/2022 10:41, David Marchand wrote:
>>> rte_bus is unnecessarily exposed in the public API/ABI.
>>> Besides, we had cases where extending rte_bus was necessary.
>>> Announce that rte_bus will be made opaque in the public API and mark
>>> associated API as internal.
>>>
>>> Signed-off-by: David Marchand <david.marchand@redhat.com>
>>> ---
>>> A RFC series of the intended changes is available at:
>>> https://patches.dpdk.org/project/dpdk/list/?series=23811&state=%2A&archive=both 
>>>
>>>
>>> ---
>>>   doc/guides/rel_notes/deprecation.rst | 6 ++++++
>>>   1 file changed, 6 insertions(+)
>>>
>> Acked-by: Kevin Laatz <kevin.laatz@intel.com>
>>

Acked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>


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

* Re: [PATCH] doc: announce marking bus object as internal
  2022-06-30  9:41 [PATCH] doc: announce marking bus object as internal David Marchand
  2022-06-30  9:54 ` Bruce Richardson
  2022-07-04 14:29 ` Kevin Laatz
@ 2022-07-15 16:16 ` Thomas Monjalon
  2 siblings, 0 replies; 7+ messages in thread
From: Thomas Monjalon @ 2022-07-15 16:16 UTC (permalink / raw)
  To: David Marchand; +Cc: dev, Ray Kinsella

30/06/2022 11:41, David Marchand:
> rte_bus is unnecessarily exposed in the public API/ABI.
> Besides, we had cases where extending rte_bus was necessary.
> Announce that rte_bus will be made opaque in the public API and mark
> associated API as internal.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>

    Acked-by: Bruce Richardson <bruce.richardson@intel.com>
    Acked-by: Thomas Monjalon <thomas@monjalon.net>
    Acked-by: Kevin Laatz <kevin.laatz@intel.com>
    Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
    Acked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>

Applied, thanks.



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

end of thread, other threads:[~2022-07-15 16:16 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-30  9:41 [PATCH] doc: announce marking bus object as internal David Marchand
2022-06-30  9:54 ` Bruce Richardson
2022-06-30 14:36   ` Thomas Monjalon
2022-07-04 14:29 ` Kevin Laatz
2022-07-05  4:43   ` Hemant Agrawal
2022-07-05  6:59     ` Andrew Rybchenko
2022-07-15 16:16 ` Thomas Monjalon

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).