Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 04 May 1999 16:06:13 +0100
From:      Roger Hardiman <roger@cs.strath.ac.uk>
To:        Mark Murray <mark@grondar.za>
Cc:        Roger Hardiman <roger@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org, jkh@zippy.cdrom.com
Subject:   Re: cvs commit: src/sys/pccard pcic.c
Message-ID:  <372F0CE5.DB90A445@cs.strath.ac.uk>
References:  <199905041058.DAA25706@freefall.freebsd.org> <199905041254.OAA12457@greenpeace.grondar.za>

next in thread | previous in thread | raw e-mail | index | archive | help
Mark, Jordan

> It this adding functionality to 3.x brefore 4.x? If so, it should not be...

No need to worry. I should have written a change log like this:-
"Add extra sanity checking to prevent kernel panics with SMP kernels"

The change is a bug fix. SMP kernels currently panic if they contain the
pccard driver due to a problem in the pccard probe code.
The change prevents the panic occuring.


Back in December I found the problem and solved it with a 1 line fix
(7 if you include the #ifdefs and #include lines)
I posted it to -mobile and put it on my FTP site.
I should have raised a PR. Oops.

I then improved the patch and committed it to 4.x-current.
Bruce then pointed out all the places the 'improved' patch was broken.
I would fix -current now, but Warner is currently fix it for the newbus,
so there is little point until he is finished.

But after talking to Bruce, I decided to go back to the simple patch for
the 3.x commit.



Bye
Roger
--
Roger Hardiman
Strathclyde Uni Telepresence Research Group, Glasgow, Scotland.
http://telepresence.dmem.strath.ac.uk      0141 548 2897
roger@cs.strath.ac.uk


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?372F0CE5.DB90A445>