DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] ml/cnxk: remove unused memory barrier macros
@ 2024-02-07  3:36 Honnappa Nagarahalli
  2024-02-07  5:24 ` [EXT] " Srikanth Yalavarthi
  0 siblings, 1 reply; 3+ messages in thread
From: Honnappa Nagarahalli @ 2024-02-07  3:36 UTC (permalink / raw)
  To: dev, syalavarthi; +Cc: jerinj, nd, Honnappa Nagarahalli

The dmb_st abd dsb_st macros are not used in the code.
Remove them to avoid any confusion.

Signed-off-by: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>
---
 drivers/ml/cnxk/cn10k_ml_dev.h | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/drivers/ml/cnxk/cn10k_ml_dev.h b/drivers/ml/cnxk/cn10k_ml_dev.h
index cee405f3f5..ddb8b67e06 100644
--- a/drivers/ml/cnxk/cn10k_ml_dev.h
+++ b/drivers/ml/cnxk/cn10k_ml_dev.h
@@ -42,15 +42,6 @@ extern struct rte_ml_dev_ops ml_dev_dummy_ops;
 #define ML_CN10K_SP_FLAGS_OCM_NONRELOCATABLE BIT(0)
 #define ML_CN10K_SP_FLAGS_EXTENDED_LOAD_JD   BIT(1)
 
-/* Memory barrier macros */
-#if defined(RTE_ARCH_ARM)
-#define dmb_st ({ asm volatile("dmb st" : : : "memory"); })
-#define dsb_st ({ asm volatile("dsb st" : : : "memory"); })
-#else
-#define dmb_st
-#define dsb_st
-#endif
-
 struct cnxk_ml_dev;
 struct cnxk_ml_req;
 struct cnxk_ml_qp;
-- 
2.34.1


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

* RE: [EXT] [PATCH] ml/cnxk: remove unused memory barrier macros
  2024-02-07  3:36 [PATCH] ml/cnxk: remove unused memory barrier macros Honnappa Nagarahalli
@ 2024-02-07  5:24 ` Srikanth Yalavarthi
  2024-02-09  7:21   ` Jerin Jacob
  0 siblings, 1 reply; 3+ messages in thread
From: Srikanth Yalavarthi @ 2024-02-07  5:24 UTC (permalink / raw)
  To: Honnappa Nagarahalli, dev; +Cc: Jerin Jacob, nd, Srikanth Yalavarthi

> -----Original Message-----
> From: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>
> Sent: Wednesday, February 7, 2024 9:06 AM
> To: dev@dpdk.org; Srikanth Yalavarthi <syalavarthi@marvell.com>
> Cc: Jerin Jacob <jerinj@marvell.com>; nd@arm.com; Honnappa Nagarahalli
> <honnappa.nagarahalli@arm.com>
> Subject: [EXT] [PATCH] ml/cnxk: remove unused memory barrier macros
> 
> The dmb_st abd dsb_st macros are not used in the code.
> Remove them to avoid any confusion.
> 
> Signed-off-by: Honnappa Nagarahalli <mailto:honnappa.nagarahalli@arm.com>

Acked-by: Srikanth Yalavarthi <syalavarthi@marvell.com>

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

* Re: [EXT] [PATCH] ml/cnxk: remove unused memory barrier macros
  2024-02-07  5:24 ` [EXT] " Srikanth Yalavarthi
@ 2024-02-09  7:21   ` Jerin Jacob
  0 siblings, 0 replies; 3+ messages in thread
From: Jerin Jacob @ 2024-02-09  7:21 UTC (permalink / raw)
  To: Srikanth Yalavarthi; +Cc: Honnappa Nagarahalli, dev, Jerin Jacob, nd

On Wed, Feb 7, 2024 at 11:10 AM Srikanth Yalavarthi
<syalavarthi@marvell.com> wrote:
>
> > -----Original Message-----
> > From: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>
> > Sent: Wednesday, February 7, 2024 9:06 AM
> > To: dev@dpdk.org; Srikanth Yalavarthi <syalavarthi@marvell.com>
> > Cc: Jerin Jacob <jerinj@marvell.com>; nd@arm.com; Honnappa Nagarahalli
> > <honnappa.nagarahalli@arm.com>
> > Subject: [EXT] [PATCH] ml/cnxk: remove unused memory barrier macros
> >
> > The dmb_st abd dsb_st macros are not used in the code.
> > Remove them to avoid any confusion.
> >
> > Signed-off-by: Honnappa Nagarahalli <mailto:honnappa.nagarahalli@arm.com>
>
> Acked-by: Srikanth Yalavarthi <syalavarthi@marvell.com>


Applied to dpdk-next-net-mrvl/for-main. Thanks

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

end of thread, other threads:[~2024-02-09  7:21 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-07  3:36 [PATCH] ml/cnxk: remove unused memory barrier macros Honnappa Nagarahalli
2024-02-07  5:24 ` [EXT] " Srikanth Yalavarthi
2024-02-09  7:21   ` Jerin Jacob

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