2014-02-20 22:53:45

by KY Srinivasan

[permalink] [raw]
Subject: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified resources

Fix a bug in the resource walking code.

Signed-off-by: K. Y. Srinivasan <[email protected]>
---
drivers/hv/vmbus_drv.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c
index b37c91b..2352ae48 100644
--- a/drivers/hv/vmbus_drv.c
+++ b/drivers/hv/vmbus_drv.c
@@ -899,10 +899,12 @@ static acpi_status vmbus_walk_resources(struct acpi_resource *res, void *ctx)
switch (res->type) {
case ACPI_RESOURCE_TYPE_IRQ:
irq = res->data.irq.interrupts[0];
+ break;

case ACPI_RESOURCE_TYPE_ADDRESS64:
hyperv_mmio_start = res->data.address64.minimum;
hyperv_mmio_size = res->data.address64.address_length;
+ break;
}

return AE_OK;
--
1.7.4.1


2014-02-20 23:04:40

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified resources

On Thu, Feb 20, 2014 at 03:45:12PM -0800, K. Y. Srinivasan wrote:
> Fix a bug in the resource walking code.
>
> Signed-off-by: K. Y. Srinivasan <[email protected]>
> ---
> drivers/hv/vmbus_drv.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)

Is this for 3.14-final or 3.15?

I ask this every time, someday you will let me know ahead of time...

2014-02-20 23:35:48

by KY Srinivasan

[permalink] [raw]
Subject: RE: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified resources



> -----Original Message-----
> From: Greg KH [mailto:[email protected]]
> Sent: Thursday, February 20, 2014 3:06 PM
> To: KY Srinivasan
> Cc: [email protected]; [email protected]
> Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified
> resources
>
> On Thu, Feb 20, 2014 at 03:45:12PM -0800, K. Y. Srinivasan wrote:
> > Fix a bug in the resource walking code.
> >
> > Signed-off-by: K. Y. Srinivasan <[email protected]>
> > ---
> > drivers/hv/vmbus_drv.c | 2 ++
> > 1 files changed, 2 insertions(+), 0 deletions(-)
>
> Is this for 3.14-final or 3.15?
>
> I ask this every time, someday you will let me know ahead of time...

I am sorry Greg. This is for 3.14-final.

Regards,

K. Y

2014-02-20 23:45:36

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified resources

On Thu, Feb 20, 2014 at 03:45:12PM -0800, K. Y. Srinivasan wrote:
> Fix a bug in the resource walking code.
>
> Signed-off-by: K. Y. Srinivasan <[email protected]>
> ---
> drivers/hv/vmbus_drv.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)

Fails to apply to my char-misc-next branch (if you want it in
3.14-final, that's where it belongs...):
checking file drivers/hv/vmbus_drv.c
Hunk #1 FAILED at 899.
1 out of 1 hunk FAILED

2014-02-20 23:46:45

by KY Srinivasan

[permalink] [raw]
Subject: RE: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified resources



> -----Original Message-----
> From: Greg KH [mailto:[email protected]]
> Sent: Thursday, February 20, 2014 3:47 PM
> To: KY Srinivasan
> Cc: [email protected]; [email protected]
> Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified
> resources
>
> On Thu, Feb 20, 2014 at 03:45:12PM -0800, K. Y. Srinivasan wrote:
> > Fix a bug in the resource walking code.
> >
> > Signed-off-by: K. Y. Srinivasan <[email protected]>
> > ---
> > drivers/hv/vmbus_drv.c | 2 ++
> > 1 files changed, 2 insertions(+), 0 deletions(-)
>
> Fails to apply to my char-misc-next branch (if you want it in
> 3.14-final, that's where it belongs...):
> checking file drivers/hv/vmbus_drv.c
> Hunk #1 FAILED at 899.
> 1 out of 1 hunk FAILED

I will rebase it send it to you shortly.

Thanks,

K. Y

2014-02-21 00:19:31

by KY Srinivasan

[permalink] [raw]
Subject: RE: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified resources



> -----Original Message-----
> From: Greg KH [mailto:[email protected]]
> Sent: Thursday, February 20, 2014 3:47 PM
> To: KY Srinivasan
> Cc: [email protected]; [email protected]
> Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified
> resources
>
> On Thu, Feb 20, 2014 at 03:45:12PM -0800, K. Y. Srinivasan wrote:
> > Fix a bug in the resource walking code.
> >
> > Signed-off-by: K. Y. Srinivasan <[email protected]>
> > ---
> > drivers/hv/vmbus_drv.c | 2 ++
> > 1 files changed, 2 insertions(+), 0 deletions(-)
>
> Fails to apply to my char-misc-next branch (if you want it in
> 3.14-final, that's where it belongs...):
> checking file drivers/hv/vmbus_drv.c
> Hunk #1 FAILED at 899.
> 1 out of 1 hunk FAILED

Greg,

This patch is based on this commit:

commit 90f3453585479d5beb75058da46eb573ced0e6ac

Signed-off-by: K. Y. Srinivasan <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>

I just got your char-misc-next branch and it does not have the needed commit.

Regards,

K. Y

2014-02-21 01:00:27

by KY Srinivasan

[permalink] [raw]
Subject: RE: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified resources



> -----Original Message-----
> From: [email protected] [mailto:driverdev-devel-
> [email protected]] On Behalf Of KY Srinivasan
> Sent: Thursday, February 20, 2014 4:19 PM
> To: Greg KH
> Cc: [email protected]; [email protected]
> Subject: RE: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified
> resources
>
>
>
> > -----Original Message-----
> > From: Greg KH [mailto:[email protected]]
> > Sent: Thursday, February 20, 2014 3:47 PM
> > To: KY Srinivasan
> > Cc: [email protected]; [email protected]
> > Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified
> > resources
> >
> > On Thu, Feb 20, 2014 at 03:45:12PM -0800, K. Y. Srinivasan wrote:
> > > Fix a bug in the resource walking code.
> > >
> > > Signed-off-by: K. Y. Srinivasan <[email protected]>
> > > ---
> > > drivers/hv/vmbus_drv.c | 2 ++
> > > 1 files changed, 2 insertions(+), 0 deletions(-)
> >
> > Fails to apply to my char-misc-next branch (if you want it in
> > 3.14-final, that's where it belongs...):
> > checking file drivers/hv/vmbus_drv.c
> > Hunk #1 FAILED at 899.
> > 1 out of 1 hunk FAILED
>
> Greg,
>
> This patch is based on this commit:
>
> commit 90f3453585479d5beb75058da46eb573ced0e6ac
>
> Signed-off-by: K. Y. Srinivasan <[email protected]>
> Signed-off-by: Greg Kroah-Hartman <[email protected]>
>
> I just got your char-misc-next branch and it does not have the needed commit.

Greg,

The commit referenced above was checked into your char-misc.git tree on the 7th of Feb.
Can you check in this patch to the same tree. I could resend this patch.

Regards,

K. Y
>
> Regards,
>
> K. Y
> _______________________________________________
> devel mailing list
> [email protected]
> http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

2014-02-21 02:18:36

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified resources

On Fri, Feb 21, 2014 at 12:19:22AM +0000, KY Srinivasan wrote:
>
>
> > -----Original Message-----
> > From: Greg KH [mailto:[email protected]]
> > Sent: Thursday, February 20, 2014 3:47 PM
> > To: KY Srinivasan
> > Cc: [email protected]; [email protected]
> > Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified
> > resources
> >
> > On Thu, Feb 20, 2014 at 03:45:12PM -0800, K. Y. Srinivasan wrote:
> > > Fix a bug in the resource walking code.
> > >
> > > Signed-off-by: K. Y. Srinivasan <[email protected]>
> > > ---
> > > drivers/hv/vmbus_drv.c | 2 ++
> > > 1 files changed, 2 insertions(+), 0 deletions(-)
> >
> > Fails to apply to my char-misc-next branch (if you want it in
> > 3.14-final, that's where it belongs...):
> > checking file drivers/hv/vmbus_drv.c
> > Hunk #1 FAILED at 899.
> > 1 out of 1 hunk FAILED
>
> Greg,
>
> This patch is based on this commit:
>
> commit 90f3453585479d5beb75058da46eb573ced0e6ac
>
> Signed-off-by: K. Y. Srinivasan <[email protected]>
> Signed-off-by: Greg Kroah-Hartman <[email protected]>
>
> I just got your char-misc-next branch and it does not have the needed commit.

Then I guess this isn't 3.14-final material, right?

Please resend it when you have determined which branch it should go
into, it's out of my queue now.

greg k-h

2014-02-21 03:10:49

by KY Srinivasan

[permalink] [raw]
Subject: RE: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified resources



> -----Original Message-----
> From: Greg KH [mailto:[email protected]]
> Sent: Thursday, February 20, 2014 6:21 PM
> To: KY Srinivasan
> Cc: [email protected]; [email protected]
> Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI specified
> resources
>
> On Fri, Feb 21, 2014 at 12:19:22AM +0000, KY Srinivasan wrote:
> >
> >
> > > -----Original Message-----
> > > From: Greg KH [mailto:[email protected]]
> > > Sent: Thursday, February 20, 2014 3:47 PM
> > > To: KY Srinivasan
> > > Cc: [email protected]; [email protected]
> > > Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in getting ACPI
> specified
> > > resources
> > >
> > > On Thu, Feb 20, 2014 at 03:45:12PM -0800, K. Y. Srinivasan wrote:
> > > > Fix a bug in the resource walking code.
> > > >
> > > > Signed-off-by: K. Y. Srinivasan <[email protected]>
> > > > ---
> > > > drivers/hv/vmbus_drv.c | 2 ++
> > > > 1 files changed, 2 insertions(+), 0 deletions(-)
> > >
> > > Fails to apply to my char-misc-next branch (if you want it in
> > > 3.14-final, that's where it belongs...):
> > > checking file drivers/hv/vmbus_drv.c
> > > Hunk #1 FAILED at 899.
> > > 1 out of 1 hunk FAILED
> >
> > Greg,
> >
> > This patch is based on this commit:
> >
> > commit 90f3453585479d5beb75058da46eb573ced0e6ac
> >
> > Signed-off-by: K. Y. Srinivasan <[email protected]>
> > Signed-off-by: Greg Kroah-Hartman <[email protected]>
> >
> > I just got your char-misc-next branch and it does not have the needed commit.
>
> Then I guess this isn't 3.14-final material, right?
>
> Please resend it when you have determined which branch it should go
> into, it's out of my queue now.

Thanks Greg, I will resend the patch. Please apply it to char-misc.git tree.

Regards,

K. Y