Re: possible spinlock optimizations

Kurt Garloff (garloff@suse.de)
Wed, 29 Sep 1999 12:30:51 +0200


--4SFOXa2GPu3tIq4H
Content-Type: text/plain; charset=iso-8859-1
Content-Transfer-Encoding: quoted-printable

On Tue, Sep 28, 1999 at 08:28:35PM +0200, Ingo Molnar wrote:
> > ??? I am fixing nothing. The old code is not buggy.
>=20
> by 'fixing the symptom' i ment 'making the symptom to go away'. The
> symptom (the effects of spinlocks held for a long time) can indeed be
> considered an 'abstract bug'.
>=20
> > The changes doesn't impact at all the fast path. It will only decrease =
the
> > irq latency during contection with the only cost of running _a_ cli bef=
ore
> > trylock _only_ each time we exit from the slow path.
>=20
> it's not only a question of wether some patch impacts the preferred good
> case. 'good' is always relative to the bad case, and if you make the 'bad
> case' appear less bad then you've also effectively hurt the good case. We
> want the 'good case' stick out loud and clear.

No, no, no.

If you optimize something, say a file-system by having the indirect blocks
next to the data blocks, you want to make sure, that=20
(a) this happens as often as possible
(b) this fast path is executed as fast as possible
And you want to add
(c) don't optimize the slow path in order to provide motivation for people
to fix the FS further to avoid it completely

You are right, if the slow path could be avoided completely by proper desig=
n.
In the FS, like in the spinlock case, it can't!

There are 8-way (and more) machines out there and you will have a hard time
to avoid spinlock contention completely, if you are I/O bound. So why not
decrease the bad effects of spinlock contention?

--=20
Kurt Garloff <garloff@suse.de> Wuppertal, FRG
PGP2 key: See mail header, key servers Linux kernel development
SuSE GmbH, N=FCrnberg, FRG SCSI drivers: tmscsim(DC390), DC395

--4SFOXa2GPu3tIq4H
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.0 (GNU/Linux)
Comment: For info see http://www.gnupg.org

iQCVAwUBN/HqWxaQN/7O/JIVAQEwSAP+PjpjMtk8FNblIVepDQ/MRnRCkMgr4nEC
3dWBpM+BgzGMalnUBF56ChMigZR6CIlHGlEwFLNBLYqowYe1AP6LUn6Kp3vMLTRZ
utmZcuSonINALkzU9yBgJeMlbLp4npNNqW8WdMHc4UIZbVJ+LNCGf3ChVcgb22IM
mrHnG6QcOYI=
=I/C4
-----END PGP SIGNATURE-----

--4SFOXa2GPu3tIq4H--

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/