Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1169852 > unrolled thread

Re: [PATCH v2 11/15] KVM: MTRR: sort variable MTRRs

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v2 11/15] KVM: MTRR: sort variable MTRRs Paolo Bonzini <pbonzini@redhat.com> - 2015-06-22 13:30 +0200
    Re: [PATCH v2 11/15] KVM: MTRR: sort variable MTRRs Xiao Guangrong <guangrong.xiao@linux.intel.com> - 2015-06-23 04:40 +0200

#1169852 — Re: [PATCH v2 11/15] KVM: MTRR: sort variable MTRRs

On 17/06/2015 18:11, Paolo Bonzini wrote:
> Also, this loop looks weird.  Is this what you wanted?
> 
>         list_for_each_entry(tmp, &mtrr_state->head, node)
>                 if (cur->base >= tmp->base)
>                         break;
>         list_add_tail(&cur->node, &tmp->node);
> 
> If so, can you look at kvm/queue and see if it is okay for you (so that
> we can get the series in 4.2)?

Ping?

If I don't get testing results before Wednesday, I'll drop this series
from the 4.2 pull request.

Paolo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1170344

On 06/22/2015 07:24 PM, Paolo Bonzini wrote:
>
>
> On 17/06/2015 18:11, Paolo Bonzini wrote:
>> Also, this loop looks weird.  Is this what you wanted?
>>
>>          list_for_each_entry(tmp, &mtrr_state->head, node)
>>                  if (cur->base >= tmp->base)
>>                          break;
>>          list_add_tail(&cur->node, &tmp->node);
>>
>> If so, can you look at kvm/queue and see if it is okay for you (so that
>> we can get the series in 4.2)?
>
> Ping?
>
> If I don't get testing results before Wednesday, I'll drop this series
> from the 4.2 pull request.

Paolo, sorry for the delay. Your changes are good to me. Thanks!
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web