[Kos-dev] Nouvelles kwaitqueue

[Thomas Petazzoni]

This is an OpenPGP/MIME signed message (RFC 2440 and 3156)
--------------enig672C8FC0D3939732200B59AF
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit

Hello,

d2, j'ai regardé tes kwaitqueue, et il y a juste une chose qui me gène :
dans chaque kwaitqueue, il y a un lock, mais ce n'est pas les fonctions
qui manipulent les kwaitqueues qui s'occupent de ce lock.

Par exemple, si on a une sémaphore, on aura vraisemblablement au minimum :

struct sem {
  kwaitqueue_t kwq;
  spinlock_t lock;
};

Et donc quand on veut manipuler la sémaphore, il faut a la fois locker
la sémaphore, et la kwaitqueue.

 1) Ca me parait pas tres propre que semaphore a trifouiller des champs
internes à kwaitqueue.
 2) Ca sert pas à grand chose de prendre plein de locks de partout.

Bref, je me demande si kwaitqueue ne devrait pas etre completement
unsafe, quitte a faire des vraies wait queues, qui elles feront du lock.

Qu'en penses-tu ?

Thomas
-- 
PETAZZONI Thomas - thomas DOT petazzoni AT enix DOT org - UIN : 34937744
Web: http://www.enix.org/~thomas/
KOS: http://kos.enix.org/ - Lolut: http://lolut.utbm.info
Fingerprint : 0BE1 4CF3 CEA4 AC9D CC6E  1624 F653 CB30 98D3 F7A7

--------------enig672C8FC0D3939732200B59AF
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQE+3vQN9lPLMJjT96cRAvwDAJ9t26uEeRpVTYp3oIbO4nObDC0LRQCdGHbV
srHA+7NLOcqT8yikNYsHPOY=
=ZFLv
-----END PGP SIGNATURE-----

--------------enig672C8FC0D3939732200B59AF--