2016-07-01 17:11 GMT+09:00 Andrey Ryabinin <aryabinin@xxxxxxxxxxxxx>: > > > On 07/01/2016 10:53 AM, js1304@xxxxxxxxx wrote: >> From: Joonsoo Kim <iamjoonsoo.kim@xxxxxxx> >> >> If we move an item on qlist's tail, we need to update qlist's tail >> properly. curr->next can be NULL since it is singly linked list >> so it is invalid for tail. curr is scheduled to be moved so >> using prev would be correct. > > Hmm.. prev may be the element that moved in 'to' list. We need to assign the last element > from which is in ther 'from' list. You're right. Also, I find another bug on this function. I manage them on v2 and sent. Thanks. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>