Welcome to the forums. Please post in English or French.

You are not logged in.

#1 2014-05-20 22:30:00

ThTh
Member
From: Lyon, France
Registered: 2007-12-23
Posts: 42
Website

[patch] bibc/utilitai/mempid.c - /proc/pid/status

Bonsoir,

Dans bibc/utilitai/mempid.c, pour des systèmes de type POSIX, le code va lire des infos sur la mémoire dans /proc/pid/status.
Or pour FreeBSD, qui est pourtant POSIX, ça ne marche pas (et même ça segfault !) car le status n'a pas la même structure.
Et je suppose que c'est pareil pour d'autres systèmes de type UNIX...

On pourrait lire dans /compat/linux/proc/pid/status qui est compatible avec Linux, mais d'une part tous les systèmes n'activent pas le mode de compatibilité, et d'autre part ce pseudo-FS n'est pas forcément monté - entre autres il ne l'est pas sur les machines qui produisent les packages.

Je vous propose donc le patch ci-joint, qui utilise la bibliothèque kvm.

N.B.: il n'est pas tout à fait complet, car je n'ai pas trouvé la correspondance pour VmPeak, que je renseigne à -1, et ça n'a pas l'air de poser de problèmes.

Bien cordialement,
--
Th. Thomas.


Attachments:
bibc_utilitai_mempid.c.diff, Size: 1.76 KiB, Downloads: 237

--
Th. Thomas.

Offline

#2 2014-05-21 15:26:56

delmas
Administrator
From: EDF R&D
Registered: 2007-12-12
Posts: 837

Re: [patch] bibc/utilitai/mempid.c - /proc/pid/status

Merci pour cette contribution.

Pourquoi ne pas proposer ce patch en faisant un pull request sur le site bitbucket de Code_Aster.


Code_Aster release : unstable on (Ubuntu Precise Pangolin 12.04 64 bits) - GNU + Intel

Code_Aster. What else ?

Offline

#3 2014-05-21 20:27:30

mathieu.courtois
Administrator
From: France
Registered: 2007-11-21
Posts: 1,167

Re: [patch] bibc/utilitai/mempid.c - /proc/pid/status

Hello,

Tu définis __FreeBSD__ à la main ?
Le mieux serait de le faire lors du `waf configure`.
Voir le fichier principal `wscript`, fonction `check_platform()`.

Là, sur un fichier qui ne bouge presque jamais, ça s'est bien passé mais le mieux est :
- créer un fork de codeaster-src sur bitbucket.org
- ouvrir une fiche en indiquant la référence vers le commit proposé

Je n'aime pas les PR sur bitbucket, car on est obligé de les *refuser* dès qu'on veut les amender.

MC

Last edited by courtois (2014-05-21 20:50:04)


Code_Aster release : last unstable on Ubuntu 16.04 64 bits - GNU Compilers

Please do not forget to tag your first post as *SOLVED* when it is!

Offline

#4 2014-05-21 20:40:17

ThTh
Member
From: Lyon, France
Registered: 2007-12-23
Posts: 42
Website

Re: [patch] bibc/utilitai/mempid.c - /proc/pid/status

Non, pas besoin de définir __FreeBSD__, on le récupère depuis les includes, je le teste juste !

Je ne connaissais pas Bitbucket, mais je vais faire ça, merci.


--
Th. Thomas.

Offline