Re: Little bugfix

Dr. Werner Fink (werner@suse.de)
Fri, 14 Jun 1996 11:13:49 +0200

> From: Bas Mevissen <sgm@stack.urc.tue.nl>
> Date: Thu, 13 Jun 1996 21:02:55 +-200
> MIME-Version: 1.0
> Content-Type: multipart/mixed; boundary="---- =_NextPart_000_01BB597A.F1830B80"
> Sender: owner-linux-kernel@vger.rutgers.edu
> Precedence: bulk
>
>
> ------ =_NextPart_000_01BB597A.F1830B80
> Content-Type: text/plain; charset="us-ascii"
> Content-Transfer-Encoding: 7bit
>
> Hi all,
>
> Here a little bug fix for the forgotten MOD_INC/DEC_COUNT in 8390.c:
>
> [[ 8390~1.PAT : 5125 in WINMAIL.DAT ]]
> That's all,
>
> Bye,
>
> Bas.
>
>
> ------ =_NextPart_000_01BB597A.F1830B80
> Content-Type: application/ms-tnef
> Content-Transfer-Encoding: base64
>

[... BIG non-Ascii-M$-Nonsens deletet ...]

>
> ------ =_NextPart_000_01BB597A.F1830B80--
>
>

Does you patch look's like:

--- drivers/net/8390.c.orig Thu Jun 13 20:51:54 1996
+++ drivers/net/8390.c Thu Jun 13 20:51:45 1996
@@ -117,6 +117,8 @@
return -ENXIO;
}

+ MOD_INC_USE_COUNT;
+
irq2dev_map[dev->irq] = dev;
NS8390_init(dev, 1);
dev->start = 1;
@@ -127,6 +129,8 @@
/* Opposite of above. Only used when "ifconfig <devname> down" is done. */
int ei_close(struct device *dev)
{
+ MOD_DEC_USE_COUNT;
+
NS8390_init(dev, 0);
dev->start = 0;
return 0;

??

Werner