Add temporary NOTIFY logs for customer side debugging in qos branch

Review Request #695 — Created March 5, 2025 and submitted

Information
mmiriam
APV10
rel_apv_10_7_2_14_qos
AE-1230
Reviewers
pradeep, prajesh 
AE-1230 Add temporary NOTIFY logs for customer side debugging (IP header, TCP header, Ether header)




 














 

 






 
 
 

  

   

    
   

   
    
    
  • 
     
    
    • 
    
  • 
     
    
    • 
    
  • 
     
    
    • 
    
  • 
     
    
    • 
    
  • 
     
    
    • 
   

  

  

   
    

     Description
     From
     Last Updated
    

   
   



    

     
      
      
According to the documentation and general best practices for POSIX threads, you should not call pthread_mutex_init multiple times on a …

     		
     
      prajeshprajesh
     
     
      
     
    




    

     
      
      
should it be called only once?

     		
     
      prajeshprajesh
     
     
      
     
    




    

     
      
      
should we log it only for HASH_INSERT case?

     		
     
      prajeshprajesh
     
     
      
     
    


   
  

 












  

 








 














 





 

  

   




prajesh




   

   







  

 



 

  


   

   

    
 
prajesh


    
   


  

  

  



    




  1. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 

















  2. 
 
    
  
  
    
  
 

   
    
    
    



    
 
    
  
    
   
   
    
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    should it be called only once?
    
  
    
 
    

 
    
  


    
 
    
 
      

  


    1. 
 
 
      
 
      
  
 
      
 
      
  
      Fixed it by adding calling it in rirt_init()
      
 
      

      2. 


 
    


    

 
    

    3. 






  3. 
 
    
  
  
    
  
 

   
    
    
    



    
 
    
  
    
   
   
    
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    should we log it only for HASH_INSERT case?
    
  
    
 
    

 
    
  


    
 
    
 
      

  


    1. 
 
 
      
 
      
  
 
      
 
      
  
      As discussed offline we may miss logs when case is HASH_NOINSERT and still got the entry in rts table
      
 
      

      2. 


 
    


    

 
    

    4. 




  4. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    5. 




  

 
















 





 

  

   




prajesh




   

   







  

 



 

  


   

   

    
 
prajesh


    
   


  

  

  



    




  1. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 

















  2. 
 
    
  
  
    
  
 

   
    
    
    



    
 
    
  
    
   
   
    
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    According to the documentation and general best practices for POSIX threads, you should not call pthread_mutex_init multiple times on a single mutex without first destroying it. Here's a breakdown:
    

    Undefined Behavior:
    
Attempting to initialize an already initialized mutex results in undefined behavior. This means the program's outcome is unpredictable and could lead to errors, crashes, or unexpected results.   
    
Error Conditions:
    
Implementations of the Pthreads library may detect attempts to re-initialize a mutex and return an EBUSY error. However, this behavior is not guaranteed, and the system might not always catch it.
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3. 




  3. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    4. 




  

 
















 



 

  

   




mmiriam




   

   

  

 



 















 





 

  

   




prajesh




   

   







  

 



 

  


   

   

    
 
prajesh


    
   


  

  

  



    




  1. 
 
    
  
    
   
    Ship It!
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 















  2. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3. 




  

 
















 





 

  

   




pradeep




   

   







  

 



 

  


   

   

    
 
pradeep


    
   


  

  

  



    




  1. 
 
    
  
    
   
    Ship It!
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 















  2. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3. 




  

 
















 



 

  

   




mmiriam




   

   

  

 



 

  


   

   

    
 Review request changed

    
   


  

  

  






 

  Status:
  

  Closed (submitted)

  
 







 


 










  

 









     

    

   

  

  

   
   Loading...