Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 17 Feb 2007 14:54:49 +0100
From:      Max Laier <max@love2party.net>
To:        freebsd-net@freebsd.org
Cc:        Jeremie Le Hen <jeremie@le-hen.org>, "V.Chukharev" <chukharev@mail.ru>, cognet@freebsd.org
Subject:   Re: iwi leaks memory?
Message-ID:  <200702171454.57301.max@love2party.net>
In-Reply-To: <op.tnwacbn30g54sc@localhost>
References:  <op.tns7i21z0g54sc@localhost> <op.tnu03mar0g54sc@localhost> <op.tnwacbn30g54sc@localhost>

next in thread | previous in thread | raw e-mail | index | archive | help
--nextPart1253199.JX7CTxXP58
Content-Type: text/plain;
  charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

On Saturday 17 February 2007 14:27, V.Chukharev wrote:
> On Fri, 16 Feb 2007 23:10:24 +0200, V.Chukharev <chukharev@mail.ru>=20
wrote:
> > On Fri, 16 Feb 2007 19:43:17 +0200, Jeremie Le Hen=20
<jeremie@le-hen.org> wrote:
> >> Hi,
> >>
> >> cognet@ has once provided me a tiny hack to the iwi(4) driver and
> >> I never get such errors.  Maybe I'm not suffering enough UP/DOWN
> >> cycles to trigger it, but it might be worth trying it.
> >>
> >> Note that he has insisted that this is a *hack*.
> >>
> >> The patch is attached.  Please let us know if it makes things
> >> better.
> >
> > I cannot apply the patch ;-(
> > It seems it's for CURRENT, and I run STABLE.
>
> I applyed the patch regenerated by Oliver against STABLE (I did not
> recieve the message, found it on the list archive, that's the reason
> for replying to myself).
>
> Yes, the patch helps. The driver did not complain after 100 down/up
> cycles. Before the patch, it started to complain usually after 6..8
> cycles.

Okay ... this confirms that this is a problem with memory fragmentation. =20
It seems that keeping the DMA'able memory around is indeed the best thing=20
we can do.  As long as the firmware is as (un)reliable as it is now there=20
is no way around the casual reset.  For normal operation it seems best to=20
store the bss firmware in the DMA'able memory and keep it there.  That=20
lets us firmware_put() the image itself.  I will have a patch later=20
today, unless somebody disagrees and/or has a better idea.

=2D-=20
/"\  Best regards,                      | mlaier@freebsd.org
\ /  Max Laier                          | ICQ #67774661
 X   http://pf4freebsd.love2party.net/  | mlaier@EFnet
/ \  ASCII Ribbon Campaign              | Against HTML Mail and News

--nextPart1253199.JX7CTxXP58
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (FreeBSD)

iD8DBQBF1wkxXyyEoT62BG0RAh6SAJ9gdR9Q94z5AgFfeVdaXr7mljR1YQCeJpAX
5jmQdiUpGWv5ukNgC7eZXtA=
=tCrH
-----END PGP SIGNATURE-----

--nextPart1253199.JX7CTxXP58--



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