aic7* claims all checked EISA io ranges (was: [MAILER-DAEMON@rumms.uni-mannheim.de: Returned mail: see transcript for details])

aic7* claims all checked EISA io ranges (was: [MAILER-DAEMON@rumms.uni-mannheim.de: Returned mail: see transcript for details])

Post by Robert Schiel » Wed, 09 Apr 2003 09:30:13



Hmm, vger does not like the driver name of the Adaptec driver. ;-)

Robert


[...]

Action: failed
Status: 5.7.1
Remote-MTA: DNS; vger.kernel.org
Diagnostic-Code: SMTP; 550 5.7.1 The Triple-X in subject is way too often associated with junk email, please rephrase. ; S263532AbTDHGof
Last-Attempt-Date: Tue, 8 Apr 2003 08:56:12 +0200 (MEST)

Date: Tue, 8 Apr 2003 08:55:41 +0200






Subject: [PATCH] aic7xxx claims all checked EISA io ranges
User-Agent: Mutt/1.4i

Hello.

Some days ago a bug was introduced in aic7xxx by applying the aic7xxx driver
upgrade to both the 2.4 and the 2.5 tree.  This bug makes aic7xxx to claim all
the ioport ranges that he checks while scanning for EISA cards.  Takashi Iwai
and me iterated over the sources to finally find that the result value check
of the request_region() is negated.

Because of that the following patch goes to Justin to fix it in driver
development tree, to the official tree maintainers to fix it there, and to
Alan, because you sent this driver update to Marcelo, so I assume the same
problem is in your tree.

The patch applies to both, the 2.4 and the 2.5 tree.

And now for the trivial but obvious patch:

--- linux/drivers/scsi/aic7xxx/aic7770_osm.c    1 Apr 2003 19:57:24 -0000       1.4

                        continue;
                request_region(eisaBase, AHC_EISA_IOSIZE, "aic7xxx");
 #else
-               if (request_region(eisaBase, AHC_EISA_IOSIZE, "aic7xxx") != 0)
+               if (request_region(eisaBase, AHC_EISA_IOSIZE, "aic7xxx") == 0)
                        continue;
 #endif

Robert

--
Robert Schiele                  Tel.: +49-621-181-2517

----- End forwarded message -----

--
Robert Schiele                  Tel.: +49-621-181-2517

  application_pgp-signature_part
< 1K Download
 
 
 

aic7* claims all checked EISA io ranges (was: [MAILER-DAEMON@rumms.uni-mannheim.de: Returned mail: see transcript for details])

Post by Justin T. Gibb » Wed, 09 Apr 2003 22:40:43


Quote:> Hello.

> Some days ago a bug was introduced in aic7xxx by applying the aic7xxx driver
> upgrade to both the 2.4 and the 2.5 tree.

This particular defect was corrected in all versions of the aic7xxx
driver released by me after March 9th.  See the driver CHANGELOG in
the driver source distribution for details.

http://people.FreeBSD.org/~gibbs/linux/SRC/

--
Justin

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

 
 
 

aic7* claims all checked EISA io ranges (was: [MAILER-DAEMON@rumms.uni-mannheim.de: Returned mail: see transcript for details])

Post by Robert Schiel » Wed, 09 Apr 2003 23:00:40



> > Hello.

> > Some days ago a bug was introduced in aic7xxx by applying the aic7xxx driver
> > upgrade to both the 2.4 and the 2.5 tree.

> This particular defect was corrected in all versions of the aic7xxx
> driver released by me after March 9th.  See the driver CHANGELOG in
> the driver source distribution for details.

> http://people.FreeBSD.org/~gibbs/linux/SRC/

Thanks for your note.  Hope you didn't feel offended.  At least this was not
my intention.  I just wanted to notify all people related to the affected
driver.

So it's up to the kernel tree maintainers to bring this fix into their trees.

Robert

--
Robert Schiele                  Tel.: +49-621-181-2517

  application_pgp-signature_part
< 1K Download
 
 
 

aic7* claims all checked EISA io ranges (was: [MAILER-DAEMON@rumms.uni-mannheim.de: Returned mail: see transcript for details])

Post by Alan Co » Wed, 09 Apr 2003 23:20:27



Quote:> Thanks for your note.  Hope you didn't feel offended.  At least this was not
> my intention.  I just wanted to notify all people related to the affected
> driver.

> So it's up to the kernel tree maintainers to bring this fix into their trees.

Maintainers submit changes to the Linux kernel tree, not vice versa. Its
push not pull

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

 
 
 

aic7* claims all checked EISA io ranges (was: [MAILER-DAEMON@rumms.uni-mannheim.de: Returned mail: see transcript for details])

Post by Justin T. Gibb » Wed, 09 Apr 2003 23:40:15



>> Thanks for your note.  Hope you didn't feel offended.  At least this was not
>> my intention.  I just wanted to notify all people related to the affected
>> driver.

>> So it's up to the kernel tree maintainers to bring this fix into their trees.

> Maintainers submit changes to the Linux kernel tree, not vice versa. Its
> push not pull

As far as the 2.4.X series is concerned, pushing has not helped.  I've
seen spelling fixes and incorrorct changes get accepted from non
maintainers "instantly", while the maintainers changes are not accepted.
Considering how long it took for the last set of driver changes to make
it from -ac into kernel.org, I just assumed that this strategy was
also failing.  Is that really the only way to get updates into Marcelo's
tree?

--
Justin

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

 
 
 

aic7* claims all checked EISA io ranges (was: [MAILER-DAEMON@rumms.uni-mannheim.de: Returned mail: see transcript for details])

Post by Robert Schiel » Thu, 10 Apr 2003 08:10:08




> > Thanks for your note.  Hope you didn't feel offended.  At least this was not
> > my intention.  I just wanted to notify all people related to the affected
> > driver.

> > So it's up to the kernel tree maintainers to bring this fix into their trees.

> Maintainers submit changes to the Linux kernel tree, not vice versa. Its
> push not pull

Well, the reason why I sent the patch to both, the maintainer of the driver
and the maintainers of the trees, is that the maintainers of the trees have
the opportunity to fix one of the more important drivers before they release a
new "official" version that is broken here, and I think that the change is
quite obvious.

To clarify this: I don't care whether and when the fix goes to the official
trees, because I have the fix for my personal kernel builds and Hubert Mantel
has it for the SuSE builds, so it Works-For-Me(TM).

Robert

--
Robert Schiele                  Tel.: +49-621-181-2517

  application_pgp-signature_part
< 1K Download
 
 
 

1. aic7* claims all checked EISA io ranges (was: [MAILER-DAEMON@rumms.u

And your code goes for long periods of time without merging good fixes,
like this one (from 2.4.20):

--- aic7-20030310/drivers/scsi/aic7xxx/aic7770.c.orig   Mon Mar 10 20:46:36 2003

        if (bootverbose)
                printf("%s: Reading SEEPROM...", ahc_name(ahc));
        have_seeprom = ahc_read_seeprom(&sd, (uint16_t *)sc,
-                                       /*start_addr*/0, sizeof(sc)/2);
+                                       /*start_addr*/0, sizeof(*sc)/2);

        if (have_seeprom) {

--
 Chuck
 Can't we all just get along?
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

2. What Hardware should I buy?

3. aic7* claims all checked EISA io ranges (was: [MAILER-DAEMON@rumms

4. Newbie Question, identifying Graphic ccard

5. aic7* claims all checked EISA io ranges (was: [MAILER-DAEMON@rumms.

6. REQ: copy of hostcvt to converrt hosts.conf to named

7. aic7* claims all checked EISA io ranges

8. AIFF sound files

9. To: davidel@xmailserver.org [Fwd: Returned mail: see transcript for details]

10. Returned mail: see transcript for details

11. Michael.Will@student.uni-tuebingen.de

12. memory range - IO range

13. isapnp error - ID range check failed for 32 bytes of IO at