[PATCH v1 15/22] chnet: node: Don't use static counter for data ID

[Ammar Faizi <ammarfaizi2@gnuweeb.org>]

The calculation will be wrong if we have multiple rings. Just provide
a counter for each ring, don't share it across multiple rings.

Fixes: 34c16780ba00890faa7139092965537ef713d853 ("chnet: node: Add set_user_data support on SQE")
Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>
---
 chnet/chnet_node.cc | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/chnet/chnet_node.cc b/chnet/chnet_node.cc
index 1269577..621d7aa 100644
--- a/chnet/chnet_node.cc
+++ b/chnet/chnet_node.cc
@@ -6,10 +6,12 @@ public:
 	inline NodeCHNetRing(uint32_t entry):
 		ring_(entry)
 	{
+		ucount_.store(0, std::memory_order_relaxed);
 	}
 
 	CNRingCtx		ring_;
 	Napi::ObjectReference	ref_;
+	std::atomic<uint32_t>	ucount_;
 };
 
 class NodeCNRingSQE {
@@ -122,7 +124,6 @@ static void CHN_SQEPrepRead(const Napi::CallbackInfo &info)
 
 static Napi::Value CHN_SQESetUserData(const Napi::CallbackInfo &info)
 {
-	static std::atomic<uint32_t> count;
 	Napi::Env env = info.Env();
 	struct NodeSQData *data;
 	NodeCNRingSQE *nsqe;
@@ -135,7 +136,7 @@ static Napi::Value CHN_SQESetUserData(const Napi::CallbackInfo &info)
 	nsqe = (NodeCNRingSQE *)info.Data();
 	data = new struct NodeSQData;
 	data->ring_ = nsqe->ring_;
-	data->id_   = count++ & nsqe->ring_->ring_.cq_mask_;
+	data->id_   = nsqe->ring_->ucount_++ & nsqe->ring_->ring_.cq_mask_;
 	nsqe->ring_->ref_.Get("__udata")
 		.As<Napi::Object>()[data->id_] = info[0];
 	nsqe->sqe_->SetUserDataPtr(data);
-- 
Ammar Faizi