On 3/21/20 8:31 PM, Tetsuo Handa wrote: > On 2020/03/22 3:49, Waiman Long wrote: >> + do { >> + if (ret > key_data_len) { >> + if (unlikely(key_data)) >> + __kvzfree(key_data, key_data_len); >> + key_data_len = ret; >> + continue; /* Allocate buffer */ > Excuse me, but "continue;" inside "do { ... } while (0);" means "break;" > because "while (0)" is evaluated before continuing the loop. You are right. My mistake. Will send out a new one for patch 2. -Longman