All of lore.kernel.org
 help / color / mirror / Atom feed
* Build failures in 4.2-stable-queue
@ 2015-09-23 18:39 Guenter Roeck
  2015-09-23 20:35 ` Daniel Axtens
  0 siblings, 1 reply; 4+ messages in thread
From: Guenter Roeck @ 2015-09-23 18:39 UTC (permalink / raw)
  To: stable, Greg Kroah-Hartman; +Cc: Daniel Axtens

Building powerpc:defconfig ... failed
Building powerpc:allmodconfig ... failed

drivers/misc/cxl/pci.c: In function 'cxl_reset':
drivers/misc/cxl/pci.c:857:7: error: 'i' undeclared (first use in this function)
   for (i = 0; i < adapter->slices; i++) {
        ^

Commit 'cxl: Remove racy attempt to force EEH invocation in reset' removes a variable
which is still in use. There may be other patches missing (?). Copying the author.

Guenter

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Build failures in 4.2-stable-queue
  2015-09-23 18:39 Build failures in 4.2-stable-queue Guenter Roeck
@ 2015-09-23 20:35 ` Daniel Axtens
  2015-09-23 21:56   ` Greg Kroah-Hartman
  0 siblings, 1 reply; 4+ messages in thread
From: Daniel Axtens @ 2015-09-23 20:35 UTC (permalink / raw)
  To: Guenter Roeck, stable, Greg Kroah-Hartman

Hi,

It looks like you're missing 4e1efb403c1c ("cxl: Don't remove
AFUs/vPHBs in cxl_reset").

The patch will be fine in stable: 4.3 added EEH, and as I said in the
commit message for that patch, it is correct whether EEH is present or
not.

Regards,
Daniel


On Wed, 2015-09-23 at 11:39 -0700, Guenter Roeck wrote:
> Building powerpc:defconfig ... failed
> Building powerpc:allmodconfig ... failed
> 
> drivers/misc/cxl/pci.c: In function 'cxl_reset':
> drivers/misc/cxl/pci.c:857:7: error: 'i' undeclared (first use in
> this function)
>    for (i = 0; i < adapter->slices; i++) {
>         ^
> 
> Commit 'cxl: Remove racy attempt to force EEH invocation in reset'
> removes a variable
> which is still in use. There may be other patches missing (?).
> Copying the author.
> 
> Guenter

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Build failures in 4.2-stable-queue
  2015-09-23 20:35 ` Daniel Axtens
@ 2015-09-23 21:56   ` Greg Kroah-Hartman
  2015-09-24  3:31     ` Guenter Roeck
  0 siblings, 1 reply; 4+ messages in thread
From: Greg Kroah-Hartman @ 2015-09-23 21:56 UTC (permalink / raw)
  To: Daniel Axtens; +Cc: Guenter Roeck, stable

On Thu, Sep 24, 2015 at 06:35:54AM +1000, Daniel Axtens wrote:
> Hi,
> 
> It looks like you're missing 4e1efb403c1c ("cxl: Don't remove
> AFUs/vPHBs in cxl_reset").
> 
> The patch will be fine in stable: 4.3 added EEH, and as I said in the
> commit message for that patch, it is correct whether EEH is present or
> not.

Thanks, I've queued that one up now.

greg k-h

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Build failures in 4.2-stable-queue
  2015-09-23 21:56   ` Greg Kroah-Hartman
@ 2015-09-24  3:31     ` Guenter Roeck
  0 siblings, 0 replies; 4+ messages in thread
From: Guenter Roeck @ 2015-09-24  3:31 UTC (permalink / raw)
  To: Greg Kroah-Hartman, Daniel Axtens; +Cc: stable

On 09/23/2015 02:56 PM, Greg Kroah-Hartman wrote:
> On Thu, Sep 24, 2015 at 06:35:54AM +1000, Daniel Axtens wrote:
>> Hi,
>>
>> It looks like you're missing 4e1efb403c1c ("cxl: Don't remove
>> AFUs/vPHBs in cxl_reset").
>>
>> The patch will be fine in stable: 4.3 added EEH, and as I said in the
>> commit message for that patch, it is correct whether EEH is present or
>> not.
>
> Thanks, I've queued that one up now.
>

Confirmed, that solved the problem.

Guenter



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-09-24  3:31 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-23 18:39 Build failures in 4.2-stable-queue Guenter Roeck
2015-09-23 20:35 ` Daniel Axtens
2015-09-23 21:56   ` Greg Kroah-Hartman
2015-09-24  3:31     ` Guenter Roeck

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.