looks good for commit

regards
-steve

On Mon, 2009-05-04 at 17:31 +0200, Jan Friesse wrote:
> Iovectorized version of previous patch. Alloca + memcpy is still there,
> but only for lib_event_data structure, not for led_body.
> 
> Regards,
>   Honza
> 
> Steven Dake wrote:
> > This patch looks pretty good but it copies via alloca+memcpy the entire
> > event data buffer which doesn't change.  Is it possible to send it as an
> > iovector to avoid the extra copy?
> > If not, go ahead and merge, but the memcpy of event buffer data is somewhat
> > detrimental to performance.
> > 
> > Regards
> > -steve
> > 
> > On Thu, Apr 30, 2009 at 4:08 AM, Jan Friesse <[email protected]> wrote:
> > 
> >> Sorry, bad patch included in last mail.
> >>
> >>
> >> Jan Friesse wrote:
> >>
> >>> This patch fixes evt.c, so now no warnings are there.
> >>>
> >>> Regards,
> >>>  Honza
> >>>
> >>>
> >> _______________________________________________
> >> Openais mailing list
> >> [email protected]
> >> https://lists.linux-foundation.org/mailman/listinfo/openais
> >>
> > 
> 
> _______________________________________________
> Openais mailing list
> [email protected]
> https://lists.linux-foundation.org/mailman/listinfo/openais

_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to