Hi Christoph, On 1/9/17, 5:13 AM, "Christoph Hellwig" <hch@xxxxxxxxxxxxx> wrote: >s/Cleanup/Remove/ ? > >Maybe add a little blurb on why we even had this huge pile of dead code >despite not using selective retransmissions, and why you don't plan to >use it any time soon. > >Otherwise looks fine: > >Reviewed-by: Christoph Hellwig <hch@xxxxxx> Thanks for review. During initial implementation, tape support was included but not enabled by default on target. So far, we don’t see any target customers Requesting this support. Since this code is not being used actively we want to remove it and we will add back if there are any request in future for SRR Support Thanks, Himanshu > ��.n��������+%������w��{.n�����{������ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f