Gestion de la mémoire douteux dans un thread

Version imprimable

Bonjour,

Je démarre un thread qui se termine seulement a la fin du programme dont le but est de checker si un périphérique est connecté via usb -> HID.

J'ai tenté de 2 manières:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
          while (true)
            {
 
                if (HidDevices.IsConnected("\\\\?\\hid#vid_1b91&pid_000a#6&1d66a16e&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}"))
                {
                   ctrl.getIhm().Invoke(ctrl.getIhm().UploadDelegate, new object[] { true });
                   ctrl.getIhm().Invoke(ctrl.getIhm().EtatBarDelegate, new object[] { "Ewoo connected" });
                }
                else {
                  ctrl.getIhm().Invoke(ctrl.getIhm().UploadDelegate, new object[] { false });
                  ctrl.getIhm().Invoke(ctrl.getIhm().EtatBarDelegate, new object[] { "Ewoo no connected" });
                }                                        
                Thread.Sleep(800);
            }

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
            while (true)
            {
                HidDevices[] devices = HidDevices.Enumerate(0x1B91, 0x000A);
                if (devices.Length > 0)
                {
                   ctrl.getIhm().Invoke(ctrl.getIhm().UploadDelegate, new object[] { true });
                   ctrl.getIhm().Invoke(ctrl.getIhm().EtatBarDelegate, new object[] { "Ewoo connected" });
                }
                else {
                  ctrl.getIhm().Invoke(ctrl.getIhm().UploadDelegate, new object[] { false });
                  ctrl.getIhm().Invoke(ctrl.getIhm().EtatBarDelegate, new object[] { "Ewoo no connected" });
                }                                        
                Thread.Sleep(800);
            }

Les 2 fonctionnes cependant le programme ne fait qu'augmenter en RAM. Il est possible qu'il y ait un pb avec la libraire que j'utilise: http://labs.mike-obrien.net/view.asp...tid=hidlibrary

Est ce que qqu peut m'aider ?

30/04/2007, 13h01
Jérôme Lambert

Il vient d'où ton ctrl ?
30/04/2007, 14h19
kollyv

J'ai créé une petite structure MVC. C'est pas très important, j'ai laissé la ligne pour montrer qu'il se passe un peu qqc.
Ce qui pose problème c'est la l'accès au périphérique USB. Mais je ne vois pas bien comment faire autrement pour checker en permanence si'il est connecté
30/04/2007, 14h49
Jérôme Lambert

Dans ce cas, je n'ai observé aucun problème de mémoire car j'ai testé ton code sans la partie des ctrl et je ne vois rien aucune perte de mémoire. :?
30/04/2007, 15h22
kollyv

Si j'enlève les méthode invoke j'ai toujours le meme problème. Par contre un truc très étrange:

si je test l'application sur mon PC j'arrive a 4kb environ tout les 3 ou 4 sec et si je fais le meme test sur un autre pc j'obitent des + 2, 3 Mo par seconde 8O

Je sais pas, si tu veux jeter un coup d'oeil a l'application je peux te l'envoyer. Elle n'est pas très grande.
30/04/2007, 15h49
Desboys

Bonjour,
je ne suis pas développeur C#, mais la question qui m'intrigue c'est:
Si tu es tenu d'utiliser des tableaux "constants" dans tes "Invoke", pourquoi ne pas les déclarer une bonne fois pour toute? ça t'évite des allocations qui pourrait être non contrôlées, et à la limite pas de coût de construction toutes les 800 millisecondes.
Parce que selon moi, à chaque tour de boucle, le new ... va allouer de la mémoire, même si c'est pas grand chose.

S. Desbois
30/04/2007, 15h57
kollyv
J'aurais pas vraiement du laisser les invoke. Le problème n'est pas la je viens de tester sans ces lignes et c'est pareil. Le problème provient de cette ligne:
Code:

1 2 HidDevices[] devices = HidDevices.Enumerate(0x1B91, 0x000A);
A chaque fois j'obtiens dans la sortie la ligne: "Enumerate". J'imagine en fait que dans la librairie des objets sont créés et pas détuit.

Je viens remarquer qqc, si j'utilise cette manière pour tester la connection:
Code:

1 2 HidDevices.IsConnected("\\\\?\\hid#vid_1b91&pid_000a#6&1d66a16e&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}")
Il n'y pas d'ajour si l'appareil n'est pas connecté. Sinon c'est pareil
30/04/2007, 15h58
SaumonAgile

Citation:

Envoyé par alakazam

Bonjour,
je ne suis pas développeur C#, mais la question qui m'intrigue c'est:
Si tu es tenu d'utiliser des tableaux "constants" dans tes "Invoke", pourquoi ne pas les déclarer une bonne fois pour toute? ça t'évite des allocations qui pourrait être non contrôlées, et à la limite pas de coût de construction toutes les 800 millisecondes.
Parce que selon moi, à chaque tour de boucle, le new ... va allouer de la mémoire, même si c'est pas grand chose.

S. Desbois

Normalement le ramasse-miettes est capable de gérer cette situation, mais ça vaut peut-être le coût de vérifier, je suis d'accord.
30/04/2007, 17h42
Jérôme Lambert
Voila, je viens de vérifier ton code.

Les objets HidDevices consomment des ressources, il faut donc appeler la méthode Dispose de chacun d'eux.

Voila ce que tu peux mettre après ton if/else :
Code:

1 2 foreach (HidDevice Hid in devices) Hid.Dispose();
30/04/2007, 20h09
kollyv

Ca fonctionne chez toi? Ca n'a pas corrigé le pb pour moi :(

Il faudrait pouvoir forcer la suppression de l'objet. J'ai chercher un peu mais n'est rien trouvé. Est ce possible de le faire?
01/05/2007, 03h20
Jérôme Lambert

Personnelement ce que j'ai constaté avec le code que j'ai rajouté, c'est que la mémoire augmente lors du début de la boucle et quand on arrive au sleep et redescend et ainsi de suite. Et donc elle n'augmente plus comme avant.
01/05/2007, 09h03
kollyv

Ca n'a pas l'air de fonctionner. Je vais changer de librairie. Celle - ci a l'air de fonctionner: http://www.codeproject.com/useritems/USB_HID.asp

Merci beaucoup du coup de main !
01/05/2007, 13h21
kollyv

Elle permet d'utiliser un évenement lorsque l'appareil se connecte ou se déconnecte.
Le problème avec l^'autre librairie venait de la librairie
01/05/2007, 15h08
Mose

Pourquoi t'appelles plein de fois GetIhm() ?
Pourquoi tu fais pas 1 variable au début de ton thread, que tu appelles 1 fois GetIhm(), et que tu bosse avec cette variable ?
01/05/2007, 16h51
kollyv

Citation:

Pourquoi t'appelles plein de fois GetIhm() ?
Pourquoi tu fais pas 1 variable au début de ton thread, que tu appelles 1 fois GetIhm(), et que tu bosse avec cette variable ?

Effectivement ca serait plus otpimisé comme ca. Je vais corriger. Mais l'erreur ne provient pas de la. Une simple boucle avec l'accès HID (dans mes tests) provoque le problème.

Je vais recoder un peu l'application avec un système d'évenement.