Hi Szymon,
On Wed, Apr 04, 2012 at 10:44:19AM +0200, Szymon Janc wrote:
> > + seq_list->list = kzalloc(sizeof(u16) * alloc_size, GFP_KERNEL);
> > + if (!seq_list->list)
> > + return -ENOMEM;
> > +
> > + seq_list->mask = alloc_size - 1;
> > + seq_list->head = L2CAP_SEQ_LIST_CLEAR;
> > + seq_list->tail = L2CAP_SEQ_LIST_CLEAR;
> > + for (i = 0; i < alloc_size; i++)
> > + seq_list->list[i] = L2CAP_SEQ_LIST_CLEAR;
>
> Could use memset for this instead of loop, and maybe use kcalloc instead of
> kzalloc to allocate array?
memset is implemented trough the loop anyway and would take 2x more
iterations.
void *memset(void *s, int c, size_t count)
{
char *xs = s;
while (count--)
*xs++ = c;
return s;
}
Best regards
Andrei Emeltchenko
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
