No. It's Ok. It is correctly adding the new address to the start of the list. Of course, it would be better to use the kernel's linked list API. regards, dan carpenter