[sheepdog] [PATCH v4] dog/vdi: bug fix for voting majority, add test case to test it

Liu Yuan namei.unix at gmail.com
Thu Apr 10 06:58:39 CEST 2014


On Thu, Apr 10, 2014 at 01:09:48PM +0900, Hitoshi Mitake wrote:
> On Tue, Apr 8, 2014 at 6:52 PM, Ruoyu <liangry at ucweb.com> wrote:
> > There is a critical error in the function vote_majority_object.
> > When all the three copies are different, the original logic still
> > choose the last one as majority.
> > The root cause is step 2 of the voting algorithm was missed.
> >
> > Signed-off-by: Ruoyu <liangry at ucweb.com>
> > ---
> >  dog/vdi.c                | 22 +++++++++++++---
> >  tests/functional/077     | 65 ++++++++++++++++++++++++++++++++++++++++++++++++
> >  tests/functional/077.out | 45 +++++++++++++++++++++++++++++++++
> >  3 files changed, 129 insertions(+), 3 deletions(-)
> 
> Looks good to me, thanks for your fix.
> Reviewed-by: Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>
> 

Applied thanks

Yuan



More information about the sheepdog mailing list