* [PATCH v4.14 backport] KVM: arm/arm64: vgic-its: Preserve the revious read from the pending table
@ 2017-12-12 16:42 Christoffer Dall
2017-12-14 20:40 ` Greg KH
0 siblings, 1 reply; 7+ messages in thread
From: Christoffer Dall @ 2017-12-12 16:42 UTC (permalink / raw)
To: stable; +Cc: Marc Zyngier, gregkh, Christoffer Dall
From: Marc Zyngier <marc.zyngier@arm.com>
Commit 64afe6e9eb4841f35317da4393de21a047a883b3 upstream.
The current pending table parsing code assumes that we keep the
previous read of the pending bits, but keep that variable in
the current block, making sure it is discarded on each loop.
We end-up using whatever is on the stack. Who knows, it might
just be the right thing...
Fixes: 33d3bc9556a7d ("KVM: arm64: vgic-its: Read initial LPI pending table")
Cc: <stable@vger.kernel.org> # 4.8
Reported-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
Reviewed-by: Christoffer Dall <christoffer.dall@linaro.org>
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
---
virt/kvm/arm/vgic/vgic-its.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/virt/kvm/arm/vgic/vgic-its.c b/virt/kvm/arm/vgic/vgic-its.c
index 547f12dc4d54..d5a8bfe78bc4 100644
--- a/virt/kvm/arm/vgic/vgic-its.c
+++ b/virt/kvm/arm/vgic/vgic-its.c
@@ -393,6 +393,7 @@ static int its_sync_lpi_pending_table(struct kvm_vcpu *vcpu)
int ret = 0;
u32 *intids;
int nr_irqs, i;
+ u8 pendmask;
nr_irqs = vgic_copy_lpi_list(vcpu, &intids);
if (nr_irqs < 0)
@@ -400,7 +401,6 @@ static int its_sync_lpi_pending_table(struct kvm_vcpu *vcpu)
for (i = 0; i < nr_irqs; i++) {
int byte_offset, bit_nr;
- u8 pendmask;
byte_offset = intids[i] / BITS_PER_BYTE;
bit_nr = intids[i] % BITS_PER_BYTE;
--
2.14.2
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH v4.14 backport] KVM: arm/arm64: vgic-its: Preserve the revious read from the pending table
2017-12-12 16:42 [PATCH v4.14 backport] KVM: arm/arm64: vgic-its: Preserve the revious read from the pending table Christoffer Dall
@ 2017-12-14 20:40 ` Greg KH
2017-12-14 20:48 ` Greg KH
0 siblings, 1 reply; 7+ messages in thread
From: Greg KH @ 2017-12-14 20:40 UTC (permalink / raw)
To: Christoffer Dall; +Cc: stable, Marc Zyngier
On Tue, Dec 12, 2017 at 05:42:31PM +0100, Christoffer Dall wrote:
> From: Marc Zyngier <marc.zyngier@arm.com>
>
> Commit 64afe6e9eb4841f35317da4393de21a047a883b3 upstream.
>
> The current pending table parsing code assumes that we keep the
> previous read of the pending bits, but keep that variable in
> the current block, making sure it is discarded on each loop.
>
> We end-up using whatever is on the stack. Who knows, it might
> just be the right thing...
>
> Fixes: 33d3bc9556a7d ("KVM: arm64: vgic-its: Read initial LPI pending table")
> Cc: <stable@vger.kernel.org> # 4.8
> Reported-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
> Reviewed-by: Christoffer Dall <christoffer.dall@linaro.org>
> Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
> Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
> ---
> virt/kvm/arm/vgic/vgic-its.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
All 4 backports queued up, many thanks for them.
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v4.14 backport] KVM: arm/arm64: vgic-its: Preserve the revious read from the pending table
2017-12-14 20:40 ` Greg KH
@ 2017-12-14 20:48 ` Greg KH
2017-12-14 20:57 ` Christoffer Dall
0 siblings, 1 reply; 7+ messages in thread
From: Greg KH @ 2017-12-14 20:48 UTC (permalink / raw)
To: Christoffer Dall; +Cc: stable, Marc Zyngier
On Thu, Dec 14, 2017 at 09:40:27PM +0100, Greg KH wrote:
> On Tue, Dec 12, 2017 at 05:42:31PM +0100, Christoffer Dall wrote:
> > From: Marc Zyngier <marc.zyngier@arm.com>
> >
> > Commit 64afe6e9eb4841f35317da4393de21a047a883b3 upstream.
> >
> > The current pending table parsing code assumes that we keep the
> > previous read of the pending bits, but keep that variable in
> > the current block, making sure it is discarded on each loop.
> >
> > We end-up using whatever is on the stack. Who knows, it might
> > just be the right thing...
> >
> > Fixes: 33d3bc9556a7d ("KVM: arm64: vgic-its: Read initial LPI pending table")
> > Cc: <stable@vger.kernel.org> # 4.8
> > Reported-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
> > Reviewed-by: Christoffer Dall <christoffer.dall@linaro.org>
> > Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
> > Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
> > ---
> > virt/kvm/arm/vgic/vgic-its.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
>
> All 4 backports queued up, many thanks for them.
Hm, no, I messed up. I'm totally confused by the emails you sent and
the "drop this" and "this is real" :)
Can you resend just the patches you want applied?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v4.14 backport] KVM: arm/arm64: vgic-its: Preserve the revious read from the pending table
2017-12-14 20:48 ` Greg KH
@ 2017-12-14 20:57 ` Christoffer Dall
2017-12-14 21:29 ` Greg KH
0 siblings, 1 reply; 7+ messages in thread
From: Christoffer Dall @ 2017-12-14 20:57 UTC (permalink / raw)
To: Greg KH; +Cc: Stable, Marc Zyngier
On Thu, Dec 14, 2017 at 9:48 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Thu, Dec 14, 2017 at 09:40:27PM +0100, Greg KH wrote:
>> On Tue, Dec 12, 2017 at 05:42:31PM +0100, Christoffer Dall wrote:
>> > From: Marc Zyngier <marc.zyngier@arm.com>
>> >
>> > Commit 64afe6e9eb4841f35317da4393de21a047a883b3 upstream.
>> >
>> > The current pending table parsing code assumes that we keep the
>> > previous read of the pending bits, but keep that variable in
>> > the current block, making sure it is discarded on each loop.
>> >
>> > We end-up using whatever is on the stack. Who knows, it might
>> > just be the right thing...
>> >
>> > Fixes: 33d3bc9556a7d ("KVM: arm64: vgic-its: Read initial LPI pending table")
>> > Cc: <stable@vger.kernel.org> # 4.8
>> > Reported-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
>> > Reviewed-by: Christoffer Dall <christoffer.dall@linaro.org>
>> > Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
>> > Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
>> > ---
>> > virt/kvm/arm/vgic/vgic-its.c | 2 +-
>> > 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> All 4 backports queued up, many thanks for them.
>
> Hm, no, I messed up. I'm totally confused by the emails you sent and
> the "drop this" and "this is real" :)
>
Yeah, I had a bad day. Sorry about the confusion.
> Can you resend just the patches you want applied?
>
Yes, all four incoming now.
Thanks!
-Christoffer
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH v4.14 backport] KVM: arm/arm64: vgic-its: Preserve the revious read from the pending table
2017-12-14 20:57 ` Christoffer Dall
@ 2017-12-14 21:29 ` Greg KH
0 siblings, 0 replies; 7+ messages in thread
From: Greg KH @ 2017-12-14 21:29 UTC (permalink / raw)
To: Christoffer Dall; +Cc: Stable, Marc Zyngier
On Thu, Dec 14, 2017 at 09:57:54PM +0100, Christoffer Dall wrote:
> On Thu, Dec 14, 2017 at 9:48 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> > On Thu, Dec 14, 2017 at 09:40:27PM +0100, Greg KH wrote:
> >> On Tue, Dec 12, 2017 at 05:42:31PM +0100, Christoffer Dall wrote:
> >> > From: Marc Zyngier <marc.zyngier@arm.com>
> >> >
> >> > Commit 64afe6e9eb4841f35317da4393de21a047a883b3 upstream.
> >> >
> >> > The current pending table parsing code assumes that we keep the
> >> > previous read of the pending bits, but keep that variable in
> >> > the current block, making sure it is discarded on each loop.
> >> >
> >> > We end-up using whatever is on the stack. Who knows, it might
> >> > just be the right thing...
> >> >
> >> > Fixes: 33d3bc9556a7d ("KVM: arm64: vgic-its: Read initial LPI pending table")
> >> > Cc: <stable@vger.kernel.org> # 4.8
> >> > Reported-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
> >> > Reviewed-by: Christoffer Dall <christoffer.dall@linaro.org>
> >> > Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
> >> > Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
> >> > ---
> >> > virt/kvm/arm/vgic/vgic-its.c | 2 +-
> >> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> All 4 backports queued up, many thanks for them.
> >
> > Hm, no, I messed up. I'm totally confused by the emails you sent and
> > the "drop this" and "this is real" :)
> >
>
> Yeah, I had a bad day. Sorry about the confusion.
>
> > Can you resend just the patches you want applied?
> >
>
> Yes, all four incoming now.
Thanks, I should have this all now queued up properly, thanks for the
resend.
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v4.14 backport] KVM: arm/arm64: vgic-its: Preserve the revious read from the pending table
@ 2017-12-14 21:04 Christoffer Dall
2017-12-14 21:16 ` Greg KH
0 siblings, 1 reply; 7+ messages in thread
From: Christoffer Dall @ 2017-12-14 21:04 UTC (permalink / raw)
To: stable; +Cc: Marc Zyngier, gregkh, Christoffer Dall
From: Marc Zyngier <marc.zyngier@arm.com>
Commit 64afe6e9eb4841f35317da4393de21a047a883b3 upstream.
The current pending table parsing code assumes that we keep the
previous read of the pending bits, but keep that variable in
the current block, making sure it is discarded on each loop.
We end-up using whatever is on the stack. Who knows, it might
just be the right thing...
Fixes: 33d3bc9556a7d ("KVM: arm64: vgic-its: Read initial LPI pending table")
Cc: <stable@vger.kernel.org> # 4.8
Reported-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
Reviewed-by: Christoffer Dall <christoffer.dall@linaro.org>
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
---
virt/kvm/arm/vgic/vgic-its.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/virt/kvm/arm/vgic/vgic-its.c b/virt/kvm/arm/vgic/vgic-its.c
index 547f12dc4d54..d5a8bfe78bc4 100644
--- a/virt/kvm/arm/vgic/vgic-its.c
+++ b/virt/kvm/arm/vgic/vgic-its.c
@@ -393,6 +393,7 @@ static int its_sync_lpi_pending_table(struct kvm_vcpu *vcpu)
int ret = 0;
u32 *intids;
int nr_irqs, i;
+ u8 pendmask;
nr_irqs = vgic_copy_lpi_list(vcpu, &intids);
if (nr_irqs < 0)
@@ -400,7 +401,6 @@ static int its_sync_lpi_pending_table(struct kvm_vcpu *vcpu)
for (i = 0; i < nr_irqs; i++) {
int byte_offset, bit_nr;
- u8 pendmask;
byte_offset = intids[i] / BITS_PER_BYTE;
bit_nr = intids[i] % BITS_PER_BYTE;
--
2.14.2
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH v4.14 backport] KVM: arm/arm64: vgic-its: Preserve the revious read from the pending table
2017-12-14 21:04 Christoffer Dall
@ 2017-12-14 21:16 ` Greg KH
0 siblings, 0 replies; 7+ messages in thread
From: Greg KH @ 2017-12-14 21:16 UTC (permalink / raw)
To: Christoffer Dall; +Cc: stable, Marc Zyngier
On Thu, Dec 14, 2017 at 10:04:24PM +0100, Christoffer Dall wrote:
> From: Marc Zyngier <marc.zyngier@arm.com>
>
> Commit 64afe6e9eb4841f35317da4393de21a047a883b3 upstream.
>
> The current pending table parsing code assumes that we keep the
> previous read of the pending bits, but keep that variable in
> the current block, making sure it is discarded on each loop.
>
> We end-up using whatever is on the stack. Who knows, it might
> just be the right thing...
>
> Fixes: 33d3bc9556a7d ("KVM: arm64: vgic-its: Read initial LPI pending table")
> Cc: <stable@vger.kernel.org> # 4.8
> Reported-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
> Reviewed-by: Christoffer Dall <christoffer.dall@linaro.org>
> Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
> Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
> ---
> virt/kvm/arm/vgic/vgic-its.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Now applied, thanks.
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-12-14 21:29 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-12 16:42 [PATCH v4.14 backport] KVM: arm/arm64: vgic-its: Preserve the revious read from the pending table Christoffer Dall
2017-12-14 20:40 ` Greg KH
2017-12-14 20:48 ` Greg KH
2017-12-14 20:57 ` Christoffer Dall
2017-12-14 21:29 ` Greg KH
-- strict thread matches above, loose matches on Subject: below --
2017-12-14 21:04 Christoffer Dall
2017-12-14 21:16 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).