Re: [PATCH] perf tools: Fix indexing for decoder packet queue

2018-05-28 Thread Arnaldo Carvalho de Melo
Em Mon, May 28, 2018 at 11:13:59AM +0800, Leo Yan escreveu: > On Fri, May 25, 2018 at 05:10:54PM -0600, Mathieu Poirier wrote: > > The tail of a queue is supposed to be pointing to the next available slot > > in a queue. In this implementation the tail is incremented before it is > > used and as

Re: [PATCH] perf tools: Fix indexing for decoder packet queue

2018-05-28 Thread Arnaldo Carvalho de Melo
Em Mon, May 28, 2018 at 11:13:59AM +0800, Leo Yan escreveu: > On Fri, May 25, 2018 at 05:10:54PM -0600, Mathieu Poirier wrote: > > The tail of a queue is supposed to be pointing to the next available slot > > in a queue. In this implementation the tail is incremented before it is > > used and as

Re: [PATCH] perf tools: Fix indexing for decoder packet queue

2018-05-28 Thread Mathieu Poirier
On 27 May 2018 at 21:13, Leo Yan wrote: > On Fri, May 25, 2018 at 05:10:54PM -0600, Mathieu Poirier wrote: >> The tail of a queue is supposed to be pointing to the next available slot >> in a queue. In this implementation the tail is incremented before it is >> used and as

Re: [PATCH] perf tools: Fix indexing for decoder packet queue

2018-05-28 Thread Mathieu Poirier
On 27 May 2018 at 21:13, Leo Yan wrote: > On Fri, May 25, 2018 at 05:10:54PM -0600, Mathieu Poirier wrote: >> The tail of a queue is supposed to be pointing to the next available slot >> in a queue. In this implementation the tail is incremented before it is >> used and as such points to the

Re: [PATCH] perf tools: Fix indexing for decoder packet queue

2018-05-27 Thread Leo Yan
On Fri, May 25, 2018 at 05:10:54PM -0600, Mathieu Poirier wrote: > The tail of a queue is supposed to be pointing to the next available slot > in a queue. In this implementation the tail is incremented before it is > used and as such points to the last used element, something that has the >

Re: [PATCH] perf tools: Fix indexing for decoder packet queue

2018-05-27 Thread Leo Yan
On Fri, May 25, 2018 at 05:10:54PM -0600, Mathieu Poirier wrote: > The tail of a queue is supposed to be pointing to the next available slot > in a queue. In this implementation the tail is incremented before it is > used and as such points to the last used element, something that has the >

[PATCH] perf tools: Fix indexing for decoder packet queue

2018-05-25 Thread Mathieu Poirier
The tail of a queue is supposed to be pointing to the next available slot in a queue. In this implementation the tail is incremented before it is used and as such points to the last used element, something that has the immense advantage of centralizing tail management at a single location and

[PATCH] perf tools: Fix indexing for decoder packet queue

2018-05-25 Thread Mathieu Poirier
The tail of a queue is supposed to be pointing to the next available slot in a queue. In this implementation the tail is incremented before it is used and as such points to the last used element, something that has the immense advantage of centralizing tail management at a single location and