From: Jag Raman <jag.raman@xxxxxxxxxx> Date: Wed, 29 Mar 2017 17:51:45 -0400 > Skip the control-only packet while processing the RX queue, for LDC > channels in UNRELIABLE mode. > > Signed-off-by: Jagannathan Raman <jag.raman@xxxxxxxxxx> > Reviewed-by: Aaron Young <aaron.young@xxxxxxxxxx> > Reviewed-by: Alexandre Chartre <alexandre.chartre@xxxxxxxxxx> > Reviewed-by: Bijan Mottahedeh <bijan.mottahedeh@xxxxxxxxxx> > Reviewed-by: Liam Merwick <liam.merwick@xxxxxxxxxx> This changes the semantics of read_nonraw(). The idea is that the caller only wants us to return if there is an error, or actual data arrives (soon). This is why the loop processes control packets and continues the loop. Now, the routine can return without an error condition even if no data is available. I think you need to justify and explain more thoroughly this semantic change in behavior. -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
