Liaison série et structure termios: problème avec mon read

**clairetj** · 13/06/2014, 11h34

Bonjour a toutes et tous, je sollicite votre aide parce que j'ai un problème que je n'arrive pas à comprendre:

Je programme une liaison série type RS485 en C avec la structure termios (protocole MODBUS), j'utilise open, write et read pour ouvrir mon port, écrire et lire dessus. Voici les codes que j'utilise:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
 
#include <fcntl.h>
#include <stdio.h>
#include <sys/stat.h>
#include <unistd.h>
 
#include <stdlib.h>
#include <sys/types.h>
#include <termios.h>
#include <stdbool.h>
 
 
 
void addCRC16(char unTableau[], int longueur)
{
	long crc16 = 0;
	int boucle1 = 0;
	int boucle2 = 0;
	unsigned char valeurCourante = 0;
	unsigned char retenue = 0;
 
	printf("DEBUG: rentree dans addCRC16\n");
	crc16 = 65535;
	for(boucle1=0; boucle1 < longueur; boucle1=boucle1+1)
	{
		valeurCourante = unTableau[boucle1];
		crc16 = crc16 ^valeurCourante;
 
		for(boucle2=0; boucle2 < 8; boucle2=boucle2+1)
		{
			retenue =crc16 % 2;
			crc16 = crc16 / 2;
			if(retenue == 1)
			{
				crc16 = crc16 ^40961;
			}
		}
	}
	unTableau[longueur] = crc16 % 256;
	unTableau[longueur+1] = crc16 / 256;
 
	printf("DEBUG: fin de addCRC16\n");
}
 
 
 
int configurationPort()
{
	int port_serie;
	struct termios options;
 
	port_serie = open("/dev/ttyUSB0", O_RDWR | O_NOCTTY | O_NDELAY);
	if(port_serie < 0)
	{
		perror("Erreur d'ouverture du port serie");
		exit(-1);
	}
	else
	{
		printf("Port serie numero %d ouvert\n", port_serie);
 
		//chargement des donnees
		tcgetattr(port_serie, &options);
		//B38400 bauds
		cfsetispeed(&options, B38400);
		cfsetospeed(&options, B38400);
		options.c_cflag |= (CLOCAL | CREAD);
		//structure en 8N1
		options.c_cflag &= ~PARENB; //pas de parité
		options.c_cflag &= ~CSTOPB; //1 bit de stop
		options.c_cflag &= ~CSIZE; //option a 0
		options.c_cflag |= CS8; // 8bits
		options.c_lflag &= ~ECHO;
		options.c_lflag &= ~ICANON;
		options.c_cc[VTIME] = 0;
		options.c_cc[VMIN] = 0;
		tcsetattr(port_serie, TCSANOW, &options);
		printf("Configuration OK structure en 8N1\n");
	}
	printf("DEBUG: fin configuration port serie\n");
	return(port_serie);
}
 
void lectureCar1(int port)			//Demande de lecture premiere partie du tableau CAR
{
	unsigned char bufferEmissionModBus[10];
	unsigned char uc_tmp1 = 8;
	unsigned char uc_i;
 
	bufferEmissionModBus[0] = 1;
	bufferEmissionModBus[1] = 4;
	bufferEmissionModBus[2] = 64;		//code appel CAR 0x40
	bufferEmissionModBus[3] = 16;		//adresse début premiere partie du tableau 0x10
	bufferEmissionModBus[4] = 0;
	bufferEmissionModBus[5] = 80;		//nombre de mots que l'on veut lire 0x50
 
	addCRC16(bufferEmissionModBus, 6);
	uc_tmp1 = write(port, &bufferEmissionModBus, 8);
 
	for(uc_i=0; uc_i < uc_tmp1; uc_i++)
	{
		printf("DEBUG: bufferEmissionModBus[%d] = %d\n", uc_i, bufferEmissionModBus[uc_i]);
	}
	printf("DEBUG: Envoie termine\n");
}
 
void lectureCar2(int port)			//Demande de lecture deuxième partie du tableau CAR
{
	unsigned char bufferEmissionModBus[10];
	unsigned char uc_tmp1 = 8;
	unsigned char uc_i;
 
	bufferEmissionModBus[0] = 1;
	bufferEmissionModBus[1] = 4;
	bufferEmissionModBus[2] = 64;		//code appel CAR 0x40
	bufferEmissionModBus[3] = 96;		//adresse début seconde partie du tableau 0x60
	bufferEmissionModBus[4] = 0;
	bufferEmissionModBus[5] = 80;		//nombre de mots que l'on veut lire 0x50
 
	addCRC16(bufferEmissionModBus, 6);
	uc_tmp1 = write(port, &bufferEmissionModBus, 8);
 
	for(uc_i=0; uc_i < uc_tmp1; uc_i++)
	{
		printf("DEBUG: bufferEmissionModBus[%d] = %d\n", uc_i, bufferEmissionModBus[uc_i]);
	}
	printf("DEBUG: Envoie termine\n");
}
 
 
char * lectureReponse(int port)
{
	int reception;
	unsigned char * uc_buffer = malloc(200*sizeof(char));
	unsigned char uc_i;
 
	printf("Lecture des donnees en cours ...\n");
	fcntl(port, F_SETFL, 10); //mode bloquant de la fonction read, si aucun caractère dispo le programme attend
 
	reception = read(port, uc_buffer, 200);
 
	if(reception == -1)
	{
		printf("DEBUG: Erreur lecture\n");
	}
	else
	{
		printf("DEBUG: Reception finie\n");
		printf("Nombre d'octets reçu = %d\n", reception);
	}
	for(uc_i=0; uc_i < reception; uc_i++)
	{
		printf("DEBUG: buffer[%d] = %d\n", uc_i, uc_buffer[uc_i]);
	}
	return(uc_buffer);
	free(uc_buffer);
}
 
 
 
int main(void)
{
	int port_serie;
	unsigned char * uc_trameValeursEntrees;
 
	port_serie = configurationPort();
 
	for(;;)
	{
		LectureCar1(port_serie);
		uc_trameValeursEntrees = lectureReponse(port_serie); //reçoit 154 octets = PAS OK
		sleep(2);
		lectureCar2(port_serie);
		uc_trameValeursEntrees = lectureReponse(port_serie); //reçoit 165 octets = OK
		sleep(2);
	}
 
}

J'utilise en parallèle de l'affichage dans mon terminal de mes buffers emis et reçu, le logiciel CommTool qui me permet d'espionner ma ligne en lisant les writes et les read. Sur CommTool, les writes sont conformes à se que je souhaite et les reads sont cohérents par rapport au nombre d'octets à lire voulu. En revanche, lorsque j'affiche le read sur mon terminal, je n'ai pas le bon nombre d'octets lus, il me manque les 11 premiers octets (donc le numéro de l'esclave, le numéro de la commande, le nombre d'octets lus et le début des octets lu).

Avec-vous une idée du critère que je pourrais faire varier ou d'une manière pour être sur d'obtenir toute ma réponse ????

Cordialement,
Clairetj

**phi1981** · 13/06/2014, 16h06

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
    return(uc_buffer);
    free(uc_buffer);

Bon je suis pas allé plus loin mais basiquement tu peux pas faire ça...
return() termine l'exécution de la fonction, donc free() ne sera jamais appelé.
T'as pas eu un avertissement du compilateur ?

Edit:

J'ajoute que faire l'inverse :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
    free(uc_buffer);
    return(uc_buffer);

serait extrêmement préjudiciable, ça voudrait dire que tu retournes un pointeur sur un buffer qui a été libéré. Cause certaine d'un méchant bug.

**clairetj** · 16/06/2014, 08h44

non je n'ai jamais eu de problème avec le compilateur concernant le free après le return. En revanche c'est sur que faire l'inverse ne me serait pas utile ^^

**clairetj** · 16/06/2014, 11h16

De plus, je viens de remarqué quelque chose de très bizarre et qui semble être la source de mon problème: si je prend cette instruction

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
bufferEmissionModBus[0] = 1;
bufferEmissionModBus[1] = 4;
bufferEmissionModBus[2] = 64;		
bufferEmissionModBus[3] = 16;		
bufferEmissionModBus[4] = 0;
bufferEmissionModBus[5] = 4;

Je ne reçois dans mon read que les deux octets du CRC16 (alors que je m'attend à avoir un octet code esclave, un octet code commande, un octet nombre d'octets lus, 8 octets lus, et 2 octets CRC16)

En revanche, si je reprend cette même instruction mais à la place de l'adresse 16, je met 20 et j'augmente (comme je veux) le nombre d'octets à lire, plus aucun problème de réception avec le read. Il semblerait donc qu'il y a un problème entre l'adresse 16 et l'adresse 19. C'est très étonnant car lorsque je regarde sur CommTools la réponse que j'obtiens sur ma ligne, toutes les trames réponses (quelque soit l'adresse initiale) sont correctes. Donc mon read n'arrive pas à capter la trame pour les adresses entre 16 et 19, étrange non ???

**phi1981** · 16/06/2014, 15h48

Bon j'ai eu le temps de regarder ton code de façon plus approfondie.

Déjà, je comprends pas pourquoi tu ouvres le port avec l'attribut O_NDELAY (non-bloquant) pour ensuite demander à le passer en mode bloquant juste avant l'appel à read().
Une façon plus propre de gérer la chose (à mon sens), consiste à utiliser un time-out avec select(). Mais je ne pense pas que ce soit l'origine de ton problème.

Tu utilises apparemment un outil qui te permet d'espionner les octets reçus sur le port série, et celui-ci affiche ce que tu t'attends à recevoir. Cela veut donc dire que ton problème ne vient pas de la requête que tu fais, mais plutôt de ta façon de lire la réponse.
Comment sais-tu qu'il "manque" les 11 premiers octets ? ne serait-ce pas plutôt les 11 derniers octets ?
read() va te retourner les octets reçus au fur et à mesure qu'ils arrivent. Ton programme ne tient pas compte de ça: il envoie la requête et lit une réponse tout de suite après. Rien ne prouve qu'elle est complète.

L'idéal serait de lire le header de la réponse, l'interpréter pour savoir combien d'octets la réponse contient, et boucler jusqu'à avoir lu la quantité spécifiée. Puis vérifier, à l'aide du CRC, que la réponse est correcte avant de l'utiliser.

Tu pourrais nous donner quelques exemples de requêtes envoyées et reçues ?

**clairetj** · 16/06/2014, 15h59

Envoyé par phi1981

Comment sais-tu qu'il "manque" les 11 premiers octets ? ne serait-ce pas plutôt les 11 derniers octets ?

Je le sais en mettant en parallèle ce que j'obtiens dans mon buffer et ce que j'obtiens dans mon logiciel espion. En partant du CRC16 (donc de la fin de mon buffer et de ma trame MODBUS) je retrouve toutes les valeurs jusqu'au moment ou j'arrive au début de mon buffer alors que je ne suis pas au début de la trame MODBUS,

je vais essayer de vous mettre un exemple, laissez-moi une dizaine de minutes

Liaison série et structure termios: problème avec mon read

C

Vue hybride

Discussions similaires

Partager

Partager