[Kos-dev] Remarques sur le commit

[Thomas Petazzoni]

Salut,

Remarques sur le commit :

- en bas de pmm.h, tu mets dans le commentaires (formate pour doxygen
  je suppose), que swap_status peut etre soit PHYS_PAGE_KERNEL_LOCKED,
  soit PHYS_PAGE_SWAPPABLE, soit PHYS_PAGE_USER_LOCKED. Je suppose que
  c'est une erreur, et que swap_status ne peut etre que
  PHYS_PAGE_NON_SWAPPABLE, PHYS_PAGE_SWAPPABLE. Bref il serait peut
  etre mieux que l'argument swap_status soit un booleen.

- le change_gpfme_swap_status me gene. effectivement on veut pouvoir
  locker/delocker une page physique (mlock, munlock), mais il faudrait
  aussi avoir un mecanisme pour proteger les pages du noyau. Certains
  seront swappables (et donc on pourra changer l'etat de
  swappabilisation entre swappable et non swappables) et d'autres
  seront non swappables a tout jamais. Avec cette fonction j'ai
  l'impression que meme si on veut que la page ne soit jamais
  swappable, y'a toujours moyen de la rendre swappable... (mais peut
  etre je me trompe).

Pas d'autres remarques concernant le commit (le reste c'est du cosmethique).

Autre remarque :

- je pense franchement qu'on devrait deplacer task/_task_kstack.c dans
  arch. On y parle de CPL0, de CPL3, etc.. Ca commence vraiment a me
  gener d'avoir un truc pareil au niveau de arch-independent.

Tu bosses sur quoi maintenant David (qu'on se marche pas sur les
pieds), je pensais commencais le locking des GPFME, mais vu que tu
bosses pas mal sur PMM en ce moment, je vais bosser sur FAT/IDE, afin
d'eviter les conflits.

Thomas

-- 
PETAZZONI Thomas - thomas.petazzoni@enix.org - UIN : 34937744
(Perso)      http://www.enix.org/~thomas/
(KOS)        http://kos.enix.org/ 
(Club LinUT) http://club-linut.enix.org