Remove redundant TSP completion tracking logic

Signed-off-by: Bartosz Dunajski <bartosz.dunajski@intel.com>

shared/source/helpers/timestamp_packet.cpp

@@ -31,7 +31,7 @@ void TimestampPacketContainer::resolveDependencies(bool clearAllDependencies) {
     std::vector<TagNodeBase *> pendingNodes;
 
     for (auto node : timestampPacketNodes) {
-        if (node->canBeReleased() || clearAllDependencies) {
+        if (clearAllDependencies) {
             node->returnTag();
         } else {
             pendingNodes.push_back(node);

shared/source/helpers/timestamp_packet.h

@@ -49,20 +49,6 @@ class TimestampPackets : public TagTypeBase {
 
     static constexpr size_t getSinglePacketSize() { return sizeof(Packet); }
 
-    bool isCompleted() const {
-        if (DebugManager.flags.DisableAtomicForPostSyncs.get() == 1) {
-            return false;
-        }
-
-        for (uint32_t i = 0; i < packetsUsed; i++) {
-            if (packets[i].contextEnd == 1) {
-                return false;
-            }
-        }
-
-        return true;
-    }
-
     void initialize() {
         for (auto &packet : packets) {
             packet.contextStart = 1u;

shared/source/utilities/tag_allocator.cpp

@@ -41,7 +41,6 @@ void TagNodeBase::returnTag() {
 
 bool TagNodeBase::canBeReleased() const {
     return (!doNotReleaseNodes) &&
-           (isCompleted()) &&
            (getImplicitGpuDependenciesCount() == getImplicitCpuDependenciesCount());
 }
 

shared/source/utilities/tag_allocator.h

@@ -61,7 +61,6 @@ class TagNodeBase : public NonCopyableOrMovableClass {
     const TagAllocatorBase *getAllocator() const { return allocator; }
 
     // TagType specific calls
-    virtual bool isCompleted() const = 0;
     virtual void assignDataToAllTimestamps(uint32_t packetIndex, void *source) = 0;
 
     virtual size_t getGlobalStartOffset() const = 0;
@@ -121,8 +120,6 @@ class TagNode : public TagNodeBase, public IDNode<TagNode<TagType>> {
 
     void assignDataToAllTimestamps(uint32_t packetIndex, void *source) override;
 
-    bool isCompleted() const override;
-
     size_t getGlobalStartOffset() const override;
     size_t getContextStartOffset() const override;
     size_t getContextEndOffset() const override;

shared/source/utilities/tag_allocator.inl

@@ -280,15 +280,6 @@ void TagNode<TagType>::assignDataToAllTimestamps(uint32_t packetIndex, void *sou
     }
 }
 
-template <typename TagType>
-bool TagNode<TagType>::isCompleted() const {
-    if constexpr (TagType::getTagNodeType() == TagNodeType::TimestampPacket) {
-        return tagForCpuAccess->isCompleted();
-    } else {
-        return true;
-    }
-}
-
 template <typename TagType>
 MetricsLibraryApi::QueryHandle_1_0 &TagNode<TagType>::getQueryHandleRef() const {
     if constexpr (TagType::getTagNodeType() == TagNodeType::HwPerfCounter) {