Re: [RFC PATCH net-next 0/8] Convert some skb_<foo> functions to void

On Tue, 2015-05-05 at 16:38 -0700, Joe Perches wrote:

Another thing that's fairly common is an skb_put followed by
a memcpy or memset.

Adding a couple of functions for those like:

void *skb_put_memcpy(struct sk_buff *skb, void *from, size_t size);
void *skb_put_memset(struct sk_buff *skb, int c, size_t size);

would reduce code size and improve performance a little.

Are you OK with that too?

Could we stop the API inflation and code churn, please.

You have little idea of the hassle this means for people like us doing
backports and code maintenance, preventing us from doing useful work,
like fixing real bugs and issues.

You know, even something very simple as the 'extern' removal in include
files had consumed days of work for us because of merge conflicts, and
reviewer time.

( commit 5c9f30236f53f3fbee170e2452dfa0e5dce4727b for example)

We understand the code as is, with regular memcpy() and memset()

These functions are fine. They were fine 20 years ago, nothing changed.

Thanks.