Re: [PATCH] ACPI, APEI, Fix use resource_size

2017-12-16 Thread Rafael J. Wysocki
On Saturday, December 16, 2017 12:56:15 PM CET Borislav Petkov wrote: > On Sat, Dec 16, 2017 at 11:43:27AM +0100, Rafael J. Wysocki wrote: > > The argument of resource_size() is (const struct resource *) and res > > is of type (struct apei_res *). > > > > I'm dropping this one. > > Yeah, us revie

Re: [PATCH] ACPI, APEI, Fix use resource_size

2017-12-16 Thread Borislav Petkov
On Sat, Dec 16, 2017 at 11:43:27AM +0100, Rafael J. Wysocki wrote: > The argument of resource_size() is (const struct resource *) and res > is of type (struct apei_res *). > > I'm dropping this one. Yeah, us reviewing this one was one big FAIL. And I tell myself everytime - build those patches f

Re: [PATCH] ACPI, APEI, Fix use resource_size

2017-12-16 Thread Rafael J. Wysocki
On Sat, Dec 16, 2017 at 11:34 AM, Rafael J. Wysocki wrote: > On Thu, Dec 14, 2017 at 10:37 AM, Borislav Petkov wrote: >> On Thu, Dec 14, 2017 at 12:07:30AM +0100, Rafael J. Wysocki wrote: >>> Which is exactly why you've got this question at all. ;-) >> >> So you're saying, you've got your minions

Re: [PATCH] ACPI, APEI, Fix use resource_size

2017-12-16 Thread Rafael J. Wysocki
On Thu, Dec 14, 2017 at 10:37 AM, Borislav Petkov wrote: > On Thu, Dec 14, 2017 at 12:07:30AM +0100, Rafael J. Wysocki wrote: >> Which is exactly why you've got this question at all. ;-) > > So you're saying, you've got your minions to do review for you - you can > lay back and play all managerial

Re: [PATCH] ACPI, APEI, Fix use resource_size

2017-12-14 Thread Borislav Petkov
On Thu, Dec 14, 2017 at 12:07:30AM +0100, Rafael J. Wysocki wrote: > Which is exactly why you've got this question at all. ;-) So you're saying, you've got your minions to do review for you - you can lay back and play all managerial now, huh? :-P > OK, let's queue it up for 4.16, then. Yeah, ju

Re: [PATCH] ACPI, APEI, Fix use resource_size

2017-12-13 Thread Rafael J. Wysocki
On Wed, Dec 13, 2017 at 4:38 PM, Borislav Petkov wrote: > On Wed, Dec 13, 2017 at 01:19:55AM +0100, Rafael J. Wysocki wrote: >> On Monday, December 11, 2017 4:44:31 PM CET Vasyl Gomonovych wrote: >> > Use resource_size function on resource object >> > Underneath __request_region set res->end to st

Re: [PATCH] ACPI, APEI, Fix use resource_size

2017-12-13 Thread Borislav Petkov
On Wed, Dec 13, 2017 at 01:19:55AM +0100, Rafael J. Wysocki wrote: > On Monday, December 11, 2017 4:44:31 PM CET Vasyl Gomonovych wrote: > > Use resource_size function on resource object > > Underneath __request_region set res->end to start + n - 1 > > Lets use resourse_size to set value properly.

Re: [PATCH] ACPI, APEI, Fix use resource_size

2017-12-12 Thread Rafael J. Wysocki
On Monday, December 11, 2017 4:44:31 PM CET Vasyl Gomonovych wrote: > Use resource_size function on resource object > Underneath __request_region set res->end to start + n - 1 > Lets use resourse_size to set value properly. > > Signed-off-by: Vasyl Gomonovych Boris, what do you think? > --- >