Subject: [aqm] Additional points as response to PIE drafts WGLC

From: Rasool Al-Saadi <[email-masked]>
Date: Tue, 22 Dec 2015 10:52:23 +0000

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