DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] doc: announce DMA configuration structure changes
@ 2025-04-16 11:05 pbhagavatula
  2025-06-24  6:22 ` [EXTERNAL] " Amit Prakash Shukla
  0 siblings, 1 reply; 6+ messages in thread
From: pbhagavatula @ 2025-04-16 11:05 UTC (permalink / raw)
  To: jerinj
  Cc: vattunuru, g.singh, sachin.saxena, hemant.agrawal, fengchengwen,
	bruce.richardson, kevin.laatz, conor.walsh, gmuthukrishn,
	vvelumuri, anatoly.burakov, dev, Pavan Nikhilesh

From: Pavan Nikhilesh <pbhagavatula@marvell.com>

Deprecate rte_dma_conf structure to allow for a more flexible
configuration of DMA devices.
The new structure will have a flags field instead of multiple
boolean fields for each feature.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
---
 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 36489f6e68..854db96c08 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -142,3 +142,9 @@ Deprecation Notices
 * bus/vmbus: Starting DPDK 25.11, all the vmbus API defined in
   ``drivers/bus/vmbus/rte_bus_vmbus.h`` will become internal to DPDK.
   Those API functions are used internally by DPDK core and netvsc PMD.
+
+* dmadev: The ``rte_dma_conf`` structure is updated to include a new field
+  ``rte_dma_conf::flags`` that should be used to configure dmadev features.
+  The existing field ``rte_dma_conf::enable_silent`` is removed and replaced
+  with the new flag ``RTE_DMA_CFG_FLAG_SILENT``, to configure silent mode
+  the flag should be set in ``rte_dma_conf::flags`` during device configuration.
-- 
2.43.0


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

* RE: [EXTERNAL] [PATCH] doc: announce DMA configuration structure changes
  2025-04-16 11:05 [PATCH] doc: announce DMA configuration structure changes pbhagavatula
@ 2025-06-24  6:22 ` Amit Prakash Shukla
  2025-07-21 17:49   ` Thomas Monjalon
  0 siblings, 1 reply; 6+ messages in thread
From: Amit Prakash Shukla @ 2025-06-24  6:22 UTC (permalink / raw)
  To: Pavan Nikhilesh Bhagavatula, Jerin Jacob
  Cc: Vamsi Krishna Attunuru, g.singh, sachin.saxena, hemant.agrawal,
	fengchengwen, bruce.richardson, kevin.laatz, conor.walsh,
	Gowrishankar Muthukrishnan, Vidya Sagar Velumuri,
	anatoly.burakov, dev, Pavan Nikhilesh Bhagavatula

Acked-by: Amit Prakash Shukla <amitprakashs@marvell.com>


From: pbhagavatula@marvell.com <pbhagavatula@marvell.com> 
Sent: Wednesday, April 16, 2025 4:36 PM
To: Jerin Jacob <jerinj@marvell.com>
Cc: Vamsi Krishna Attunuru <vattunuru@marvell.com>; g.singh@nxp.com; sachin.saxena@nxp.com; hemant.agrawal@nxp.com; fengchengwen@huawei.com; bruce.richardson@intel.com; kevin.laatz@intel.com; conor.walsh@intel.com; Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com>; Vidya Sagar Velumuri <vvelumuri@marvell.com>; anatoly.burakov@intel.com; dev@dpdk.org; Pavan Nikhilesh Bhagavatula <pbhagavatula@marvell.com>
Subject: [EXTERNAL] [PATCH] doc: announce DMA configuration structure changes

From: Pavan Nikhilesh <pbhagavatula@ marvell. com> Deprecate rte_dma_conf structure to allow for a more flexible configuration of DMA devices. The new structure will have a flags field instead of multiple boolean fields for each feature. 
ZjQcmQRYFpfptBannerStart
Prioritize security for external emails: 
Confirm sender and content safety before clicking links or opening attachments 
  https://us-phishalarm-ewt.proofpoint.com/EWT/v1/CRVmXkqW!tg3Z1f8UAnVa1M-cmd367pSy50oaG5i-erPnxhrfkp_lTFDGry0-uCQnbPxTrMJQ9QhDCdGTFh21ta0HYCWLBlk3vg5y8fISwiE1hA$  ‌ 


ZjQcmQRYFpfptBannerEnd
From: Pavan Nikhilesh <mailto:pbhagavatula@marvell.com>

Deprecate rte_dma_conf structure to allow for a more flexible
configuration of DMA devices.
The new structure will have a flags field instead of multiple
boolean fields for each feature.

Signed-off-by: Pavan Nikhilesh <mailto:pbhagavatula@marvell.com>
---
 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 36489f6e68..854db96c08 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -142,3 +142,9 @@ Deprecation Notices
 * bus/vmbus: Starting DPDK 25.11, all the vmbus API defined in
   ``drivers/bus/vmbus/rte_bus_vmbus.h`` will become internal to DPDK.
   Those API functions are used internally by DPDK core and netvsc PMD.
+
+* dmadev: The ``rte_dma_conf`` structure is updated to include a new field
+  ``rte_dma_conf::flags`` that should be used to configure dmadev features.
+  The existing field ``rte_dma_conf::enable_silent`` is removed and replaced
+  with the new flag ``RTE_DMA_CFG_FLAG_SILENT``, to configure silent mode
+  the flag should be set in ``rte_dma_conf::flags`` during device configuration.
-- 
2.43.0


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

* Re: [EXTERNAL] [PATCH] doc: announce DMA configuration structure changes
  2025-06-24  6:22 ` [EXTERNAL] " Amit Prakash Shukla
@ 2025-07-21 17:49   ` Thomas Monjalon
  2025-07-25  6:04     ` Pavan Nikhilesh Bhagavatula
  0 siblings, 1 reply; 6+ messages in thread
From: Thomas Monjalon @ 2025-07-21 17:49 UTC (permalink / raw)
  To: Pavan Nikhilesh Bhagavatula, Amit Prakash Shukla
  Cc: Jerin Jacob, dev, Vamsi Krishna Attunuru, g.singh, sachin.saxena,
	hemant.agrawal, fengchengwen, bruce.richardson, kevin.laatz,
	conor.walsh, Gowrishankar Muthukrishnan, Vidya Sagar Velumuri,
	anatoly.burakov

> Deprecate rte_dma_conf structure to allow for a more flexible
> configuration of DMA devices.
> The new structure will have a flags field instead of multiple
> boolean fields for each feature.
> 
> Signed-off-by: Pavan Nikhilesh <mailto:pbhagavatula@marvell.com>
> ---
> +* dmadev: The ``rte_dma_conf`` structure is updated to include a new field
> +  ``rte_dma_conf::flags`` that should be used to configure dmadev features.
> +  The existing field ``rte_dma_conf::enable_silent`` is removed and replaced
> +  with the new flag ``RTE_DMA_CFG_FLAG_SILENT``, to configure silent mode
> +  the flag should be set in ``rte_dma_conf::flags`` during device configuration.
> 
> Acked-by: Amit Prakash Shukla <amitprakashs@marvell.com>

There is only 1 ack.
Per our policy, it will miss the release 25.07.

You can probably do this change anyway,
and keep ABI compatibility by versioning the function.



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

* Re: [EXTERNAL] [PATCH] doc: announce DMA configuration structure changes
  2025-07-21 17:49   ` Thomas Monjalon
@ 2025-07-25  6:04     ` Pavan Nikhilesh Bhagavatula
  2025-07-26  0:55       ` fengchengwen
  0 siblings, 1 reply; 6+ messages in thread
From: Pavan Nikhilesh Bhagavatula @ 2025-07-25  6:04 UTC (permalink / raw)
  To: Thomas Monjalon, Amit Prakash Shukla
  Cc: Jerin Jacob, dev, Vamsi Krishna Attunuru, g.singh, sachin.saxena,
	hemant.agrawal, fengchengwen, bruce.richardson, kevin.laatz,
	conor.walsh, Gowrishankar Muthukrishnan, Vidya Sagar Velumuri,
	anatoly.burakov

>> Deprecate rte_dma_conf structure to allow for a more flexible
>> configuration of DMA devices.
>> The new structure will have a flags field instead of multiple
>> boolean fields for each feature.
>>
>> Signed-off-by: Pavan Nikhilesh <mailto:pbhagavatula@marvell.com>
>> ---
>> +* dmadev: The ``rte_dma_conf`` structure is updated to include a new field
>> +  ``rte_dma_conf::flags`` that should be used to configure dmadev features.
>> +  The existing field ``rte_dma_conf::enable_silent`` is removed and replaced
>> +  with the new flag ``RTE_DMA_CFG_FLAG_SILENT``, to configure silent mode
>> +  the flag should be set in ``rte_dma_conf::flags`` during device configuration.
>>
>> Acked-by: Amit Prakash Shukla <amitprakashs@marvell.com>
>
>There is only 1 ack.
>Per our policy, it will miss the release 25.07.
>
>You can probably do this change anyway,
>and keep ABI compatibility by versioning the function.

Hi Fengchengwen,

Are you ok with this change? If so please ack it so that I can work on getting
an exception from techboard to merge this without deprecation notice in 25.11.

Thanks,
Pavan.

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

* Re: [EXTERNAL] [PATCH] doc: announce DMA configuration structure changes
  2025-07-25  6:04     ` Pavan Nikhilesh Bhagavatula
@ 2025-07-26  0:55       ` fengchengwen
  2025-07-28  5:11         ` Pavan Nikhilesh Bhagavatula
  0 siblings, 1 reply; 6+ messages in thread
From: fengchengwen @ 2025-07-26  0:55 UTC (permalink / raw)
  To: Pavan Nikhilesh Bhagavatula, Thomas Monjalon, Amit Prakash Shukla
  Cc: Jerin Jacob, dev, Vamsi Krishna Attunuru, g.singh, sachin.saxena,
	hemant.agrawal, bruce.richardson, kevin.laatz, conor.walsh,
	Gowrishankar Muthukrishnan, Vidya Sagar Velumuri,
	anatoly.burakov

Acked-by: Chengwen Feng <fengchengwen@huawei.com>

On 2025/7/25 14:04, Pavan Nikhilesh Bhagavatula wrote:
>>> Deprecate rte_dma_conf structure to allow for a more flexible
>>> configuration of DMA devices.
>>> The new structure will have a flags field instead of multiple
>>> boolean fields for each feature.
>>>
>>> Signed-off-by: Pavan Nikhilesh <mailto:pbhagavatula@marvell.com>
>>> ---
>>> +* dmadev: The ``rte_dma_conf`` structure is updated to include a new field
>>> +  ``rte_dma_conf::flags`` that should be used to configure dmadev features.
>>> +  The existing field ``rte_dma_conf::enable_silent`` is removed and replaced
>>> +  with the new flag ``RTE_DMA_CFG_FLAG_SILENT``, to configure silent mode
>>> +  the flag should be set in ``rte_dma_conf::flags`` during device configuration.
>>>
>>> Acked-by: Amit Prakash Shukla <amitprakashs@marvell.com>
>>
>> There is only 1 ack.
>> Per our policy, it will miss the release 25.07.
>>
>> You can probably do this change anyway,
>> and keep ABI compatibility by versioning the function.
> 
> Hi Fengchengwen,
> 
> Are you ok with this change? If so please ack it so that I can work on getting
> an exception from techboard to merge this without deprecation notice in 25.11.
> 
> Thanks,
> Pavan.
> 


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

* Re: [EXTERNAL] [PATCH] doc: announce DMA configuration structure changes
  2025-07-26  0:55       ` fengchengwen
@ 2025-07-28  5:11         ` Pavan Nikhilesh Bhagavatula
  0 siblings, 0 replies; 6+ messages in thread
From: Pavan Nikhilesh Bhagavatula @ 2025-07-28  5:11 UTC (permalink / raw)
  To: fengchengwen, techboard, Thomas Monjalon, Amit Prakash Shukla
  Cc: Jerin Jacob, dev, Vamsi Krishna Attunuru, g.singh, sachin.saxena,
	hemant.agrawal, bruce.richardson, kevin.laatz, conor.walsh,
	Gowrishankar Muthukrishnan, Vidya Sagar Velumuri,
	anatoly.burakov

>Acked-by: Chengwen Feng <fengchengwen@huawei.com>
>

Thomas,

Now that Feng Chengwen is ok with this change, can this be merged
along with the ABI breaking changes in 25.11?
Given that techboard approves the change.
This change helps reduce ABI breakage when a new feature is added.

Thanks,
Pavan.

>On 2025/7/25 14:04, Pavan Nikhilesh Bhagavatula wrote:
>>>> Deprecate rte_dma_conf structure to allow for a more flexible
>>>> configuration of DMA devices.
>>>> The new structure will have a flags field instead of multiple
>>>> boolean fields for each feature.
>>>>
>>>> Signed-off-by: Pavan Nikhilesh <mailto:pbhagavatula@marvell.com>
>>>> ---
>>>> +* dmadev: The ``rte_dma_conf`` structure is updated to include a new field
>>>> +  ``rte_dma_conf::flags`` that should be used to configure dmadev features.
>>>> +  The existing field ``rte_dma_conf::enable_silent`` is removed and replaced
>>>> +  with the new flag ``RTE_DMA_CFG_FLAG_SILENT``, to configure silent mode
>>>> +  the flag should be set in ``rte_dma_conf::flags`` during device configuration.
>>>>
>>>> Acked-by: Amit Prakash Shukla <amitprakashs@marvell.com>
>>>
>>> There is only 1 ack.
>>> Per our policy, it will miss the release 25.07.
>>>
>>> You can probably do this change anyway,
>>> and keep ABI compatibility by versioning the function.
>>
>> Hi Fengchengwen,
>>
>> Are you ok with this change? If so please ack it so that I can work on getting
>> an exception from techboard to merge this without deprecation notice in 25.11.
>>
>> Thanks,
>> Pavan.
>>



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

end of thread, other threads:[~2025-07-28  5:11 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-04-16 11:05 [PATCH] doc: announce DMA configuration structure changes pbhagavatula
2025-06-24  6:22 ` [EXTERNAL] " Amit Prakash Shukla
2025-07-21 17:49   ` Thomas Monjalon
2025-07-25  6:04     ` Pavan Nikhilesh Bhagavatula
2025-07-26  0:55       ` fengchengwen
2025-07-28  5:11         ` Pavan Nikhilesh Bhagavatula

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