Paolo Bonzini <pbonzini@xxxxxxxxxx> writes: > Il 22/04/2014 12:25, Bandan Das ha scritto: >>>> >> + if (j < i) >>>> >> + shadow_read_write_fields[j] = >>>> >> + shadow_read_write_fields[i]; >>>> >> + j++; >>> > >>> > ... you need to respin anyway because the j++ is wrong. It should be >>> > inside the "if". If you prefer, you can put it in the lhs of the >> j++ outside the "if" looks right to me. Can you please explain why >> you think it shouldn't be that way ? >> > > The way you wrote it, j will always be equal to i. Right, and that's what we want unless we come across an unsupported field. We then overwrite the unsupported field with the next field supported. j this way keeps track of the "real" length. > Paolo -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
