Re: [Qemu-devel] [PATCH 1/8] ipmi: fix SDR length value

2016-01-12 Thread Cédric Le Goater
On 01/08/2016 09:20 PM, Corey Minyard wrote: > On 01/06/2016 02:14 AM, Cédric Le Goater wrote: >> On 01/05/2016 08:59 PM, Eric Blake wrote: >>> On 01/05/2016 10:29 AM, Cédric Le Goater wrote: >>> >>> [meta-comment] Your messages were not marked in-reply-to: the 0/8 cover >>> letter, but came

Re: [Qemu-devel] [PATCH 1/8] ipmi: fix SDR length value

2016-01-08 Thread Corey Minyard
On 01/06/2016 02:14 AM, Cédric Le Goater wrote: On 01/05/2016 08:59 PM, Eric Blake wrote: On 01/05/2016 10:29 AM, Cédric Le Goater wrote: [meta-comment] Your messages were not marked in-reply-to: the 0/8 cover letter, but came through as separate threads. This makes it harder to follow,

Re: [Qemu-devel] [PATCH 1/8] ipmi: fix SDR length value

2016-01-06 Thread Cédric Le Goater
On 01/05/2016 08:59 PM, Eric Blake wrote: > On 01/05/2016 10:29 AM, Cédric Le Goater wrote: > > [meta-comment] Your messages were not marked in-reply-to: the 0/8 cover > letter, but came through as separate threads. This makes it harder to > follow, especially in mail clients that sort top-level

[Qemu-devel] [PATCH 1/8] ipmi: fix SDR length value

2016-01-05 Thread Cédric Le Goater
The IPMI BMC simulator populates the SDR table with a set of initial SDRs. The length of each SDR is taken from the record itself (byte 4) which does not include the size of the header. But, the full length (header + data) is required by the sdr_add_entry() routine. Signed-off-by: Cédric Le

Re: [Qemu-devel] [PATCH 1/8] ipmi: fix SDR length value

2016-01-05 Thread Eric Blake
On 01/05/2016 10:29 AM, Cédric Le Goater wrote: [meta-comment] Your messages were not marked in-reply-to: the 0/8 cover letter, but came through as separate threads. This makes it harder to follow, especially in mail clients that sort top-level threads by most recent activity on the thread. >