2003-08-15 00:08:10

by John Cherry

[permalink] [raw]
Subject: [PATCH] fix parallel builds for aic7xxx

My compile regression scripts were getting random build failures for
aic7xxx. The two makefiles could not handle parallel build.
Occasionally they would succeed...timing dependent. The following two
patches fix this.

Part 1 - drivers/scsi/aic7xxx/Makefile
Part 2 - drivers/scsi/aic7xxx/aicasm/Makefile

John


Attachments:
part1 (0.99 kB)
part2 (762.00 B)
Download all attachments

2003-08-16 06:37:45

by Douglas Gilbert

[permalink] [raw]
Subject: Re: [PATCH] fix parallel builds for aic7xxx

While on the subject of aic7xxx Makefiles, they add the
"-Werror" flag to CFLAGS in both lk 2.4.22-rc2 and
lk 2.6.0-test3 .

This flag can be annoying if there is some messiness in
include files somewhere. In my case some mis-applied patches
in the ide headers cause otherwise harmless compiler warning.
That is until my kernel build tries to build the aic7xxx driver.

As a side note drivers/scsi/aic7xxx/Makefile seems to be the only
Makefile in lk 2.4.22-rc2 adding the "-Werror" flag. About a
dozen Makefiles add it in lk 2.6.0-test3 (mainly in the alpha
and sparc64 architecture trees).

Doug Gilbert

2003-08-16 07:15:34

by Sam Ravnborg

[permalink] [raw]
Subject: Re: [PATCH] fix parallel builds for aic7xxx

On Sat, Aug 16, 2003 at 04:36:45PM +1000, Douglas Gilbert wrote:
> While on the subject of aic7xxx Makefiles, they add the
> "-Werror" flag to CFLAGS in both lk 2.4.22-rc2 and
> lk 2.6.0-test3 .
>
> This flag can be annoying if there is some messiness in
> include files somewhere. In my case some mis-applied patches
> in the ide headers cause otherwise harmless compiler warning.
> That is until my kernel build tries to build the aic7xxx driver.

The incentive for the aic7xx maintainer is that he do detect if a
warning happens before actually shipping a patch.
Same goes for alpha and sparc64, I recall both David and Richard
wanted to achieve the same goal.

[Wondering if kbuild could do something smart here...]

During the 2.4 series Linus added -Werror to CFLAGS in the top-level
makefile, simply to put more focus on warnings. If was only momentarily
but a lot af patches showed up removing the source of the errors.
The dense output from kbuild these days makes warnings much more
visible, and if you follow John Cherry nice statistics you can see that
the number of warnings generally decrease for each new kernel release.

Sam