[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20070327.153109.91000136.davem@davemloft.net>
Date: Tue, 27 Mar 2007 15:31:09 -0700 (PDT)
From: David Miller <davem@...emloft.net>
To: acme@...stprotocols.net
Cc: netdev@...r.kernel.org
Subject: Re: [PATCH 2/2] [SK_BUFF]: Introduce
skb_copy_from_linear_data{_offset}
From: "Arnaldo Carvalho de Melo" <acme@...stprotocols.net>
Date: Tue, 27 Mar 2007 18:56:11 -0300
> On 3/27/07, David Miller <davem@...emloft.net> wrote:
> > From: Arnaldo Carvalho de Melo <acme@...stprotocols.net>
> > Date: Tue, 27 Mar 2007 16:41:48 -0300
> >
> > > +static inline void skb_move_linear_data(const struct sk_buff *skb,
> > > + const int from_offset,
> > > + const int to_offset,
> > > + const unsigned int len)
> > > +{
> > > + memcpy(skb->data + to_offset, skb->data + from_offset, len);
> > > +}
> > > +
> >
> > What do you intend to use this for? There are no users in
> > your patch, and I'd be fearful of overlapping cases (which
> > ought to be common for anything using this) which therefore
> > would require use of memmove() instead of memcpy().
> >
> > How about we just leave out interfaces until we have a use
> > case?
>
> It should be ok now, please take a look in a throwaway branch to make
> sure, sorry for the disturbance.
Where should I pull from? Please always specify this, I delete
old emails from a thread when I expect a resubmit, so I don't
even know the original URL. THanks.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists