Dear all,

As a follow up email for our feedback as a response to PIE Draft WG last call, 
we have found some editing errors in the PIE Draft and we hope by correcting 
these error the document will become clearer for the readers:

1- In section 4.2 (page 8), in the pseudo code, "qdelay_old_ = 
current_qdelay_." should be moved to a place after 
if (current_qdelay_ == 0 &&  qdelay_old_ == 0) {
            p = p*0.98;    //1- 1/64 is sufficient
        }
because qdelay_old_ will always be equal to current_qdelay.

2- In section 5.2 (page 12),"... ; in this case the previously latched value of 
depart_rate is used.", 'depart_rate' should be 'dq_rate_'.

3- In Page 19,  in enque(Packet packet), the if-condition statement has to 
conditions for PIE->burst_allowance_ one with '< 0' and another with  '<= 0' 

       if (PIE->burst_allowance_ < 0 && drop_early() == DROP
          && PIE->burst_allowance_ <= 0) {

4-In Page 21, "PIE->last_timestamp_ = now; " should be removed as ' 
last_timestamp_' is not used in the basic PIE  pseudo code.

Regards,
Rasool Al-Saadi

_______________________________________________
aqm mailing list
aqm@ietf.org
https://www.ietf.org/mailman/listinfo/aqm

Reply via email to