Re: [Qemu-devel] [PATCH] e1000e: Fix build with ust trace backend

2016-06-04 Thread Dmitry Fleytman
> On 3 Jun 2016, at 16:47 PM, Peter Maydell wrote: > > On 3 June 2016 at 12:03, Peter Maydell wrote: >> Thanks, applied to master as a build fix. > > ...which reveals another build issue: > > https://travis-ci.org/qemu/qemu/jobs/135006245 > > hw/net/e1000e.c: In function ‘e1000e_io_write’: >

Re: [Qemu-devel] [PATCH] e1000e: Fix build with ust trace backend

2016-06-03 Thread Peter Maydell
On 3 June 2016 at 12:03, Peter Maydell wrote: > Thanks, applied to master as a build fix. ...which reveals another build issue: https://travis-ci.org/qemu/qemu/jobs/135006245 hw/net/e1000e.c: In function ‘e1000e_io_write’: hw/net/e1000e.c:170:39: error: ‘idx’ may be used uninitialized in this f

Re: [Qemu-devel] [PATCH] e1000e: Fix build with ust trace backend

2016-06-03 Thread Peter Maydell
On 2 June 2016 at 20:12, Dmitry Fleytman wrote: > ust trace backend has limitation of maximum 10 > arguments per event. Traces with more arguments > cannot be compiled for this backend. > > Trace e1000e_rx_rss_ip6 introduced by previous > commits has 11 arguments and fails to compile with > ust tr

[Qemu-devel] [PATCH] e1000e: Fix build with ust trace backend

2016-06-02 Thread Dmitry Fleytman
ust trace backend has limitation of maximum 10 arguments per event. Traces with more arguments cannot be compiled for this backend. Trace e1000e_rx_rss_ip6 introduced by previous commits has 11 arguments and fails to compile with ust trace backend. This patch fixes the problem by splitting this t