[LTP] [PATCH 1/4] [COMMITTED] nfsv4/locktests: Get rid of useless comments
Cyril Hrubis
chrubis@suse.cz
Mon Aug 28 16:31:59 CEST 2017
Removes mostly the duplict french ones but also a few written in
english.
Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
---
testcases/network/nfsv4/locks/locktests.c | 92 ++-----------------------------
testcases/network/nfsv4/locks/locktests.h | 12 +---
testcases/network/nfsv4/locks/netsync.c | 19 +------
3 files changed, 6 insertions(+), 117 deletions(-)
diff --git a/testcases/network/nfsv4/locks/locktests.c b/testcases/network/nfsv4/locks/locktests.c
index 02e8ff7b8..73fca3095 100644
--- a/testcases/network/nfsv4/locks/locktests.c
+++ b/testcases/network/nfsv4/locks/locktests.c
@@ -39,7 +39,6 @@ int esclaveLecteur;
int maitreLecteur;
int esclaveEcrivain;
-/* Quel lock sera appplique par le maitre en debut de test */
/* Which lock will be applied by the master process on test startup */
int LISTE_LOCKS[] = { READLOCK, WRITELOCK,
READLOCK, WRITELOCK,
@@ -48,9 +47,7 @@ int LISTE_LOCKS[] = { READLOCK, WRITELOCK,
BYTELOCK_READ, BYTELOCK_WRITE
};
-/* Les operations que les programes esclaves essaieront de faire */
/* The operations the slave processes will try to perform */
-
int LISTE_TESTS[] = { WRONLY, WRONLY,
RDONLY, RDONLY,
READLOCK, WRITELOCK,
@@ -71,9 +68,7 @@ char *LISTE_NOMS_TESTS[] = { "WRITE ON A READ LOCK",
"WRITE LOCK THE WHOLE FILE BYTE BY BYTE"
};
-/* Resultat attendu du test - Processus */
/* List of expected test results, when slaves are processes */
-
int LISTE_RESULTATS_PROCESS[] = { SUCCES, SUCCES,
SUCCES, SUCCES,
SUCCES, ECHEC,
@@ -82,7 +77,6 @@ int LISTE_RESULTATS_PROCESS[] = { SUCCES, SUCCES,
};
/* List of expected test results, when slaves are threads */
-
int LISTE_RESULTATS_THREADS[] = { SUCCES, SUCCES,
SUCCES, SUCCES,
SUCCES, SUCCES,
@@ -107,7 +101,6 @@ int (*load) ();
struct donneesPub dp;
-/* Manipulation des tests/ noms de tests/ resultats de tests */
/* Functions to access tests/tests names/tests results*/
int testSuiv(int n)
{
@@ -129,8 +122,7 @@ int lockSuiv(int n)
return LISTE_LOCKS[n];
}
-/* Verifie si le resultat obtenu pour le test est le resultat attendu pour ce test */
-/* Verifie the test result is the expected one */
+/* Verify the test result is the expected one */
int matchResult(int r, int n)
{
@@ -141,20 +133,15 @@ int matchResult(int r, int n)
return 0;
}
-/* Incremente le nombre de process ayant valide le test */
/* Increments the number of process which have successfully passed the test */
void compteur(int r, int n)
{
TOTAL_RESULTAT_OK[n] += matchResult(r, n);
}
-/* Validation des resultats pour le lock de fichier octet par octet
- * Pour chaque octet on verifie qu'un lock de 1 octet est bien pose
- */
/* Special case for test 'lock file byte byte by byte'.
* We ensure each byte is correctly locked.
*/
-
void validationResultats(int n)
{
int i, u, l, fsize;
@@ -164,7 +151,7 @@ void validationResultats(int n)
TOTAL_RESULTAT_OK[n] = 0;
l = FALSE;
u = TRUE;
- /* Si le resultat de l'operation attendu est un succe on prevoi d'incrementer le nombre de resultats corrects */
+
/* If the expected operation result is a success, we will have to increase the number of correct results */
if (LISTE_RESULTATS[n]) {
l = TRUE;
@@ -177,7 +164,6 @@ void validationResultats(int n)
request.l_start = i;
request.l_len = 1;
fcntl(dp.fd, F_GETLK, &request);
- /* On verifie si le lock est mis ou non */
/* Ensure the lock is correctly set */
if (request.l_type != F_UNLCK)
TOTAL_RESULTAT_OK[n] += l;
@@ -186,9 +172,6 @@ void validationResultats(int n)
}
}
-/* Procedures d'initialisation */
-/* Initialisation functions */
-
int initTest()
{
@@ -207,14 +190,12 @@ struct donneesFils *initClientFork(int i)
struct donneesPriv *dpr;
struct donneesFils *df;
- /* Initialisation des champs de donnees */
/* Initialize private data fields */
dpr = malloc(sizeof(struct donneesPriv));
df = malloc(sizeof(struct donneesFils));
dpr->whoami = i;
df->dp = &dp;
df->dpr = dpr;
- /* Initialisation du tubes de synchronisation */
/* Initialize master to client pipe */
dp.lclnt[i] = malloc(sizeof(int) * 2);
if (pipe(dp.lclnt[i]) < 0) {
@@ -229,14 +210,12 @@ struct donneesFils *initClientFork(int i)
int initialise(int clnt)
{
- /* Initialisation des donnees publiques */
/* Initialize private data fields */
printf("Init\n");
dp.nclnt = clnt;
dp.lclnt = malloc(sizeof(int *) * clnt);
dp.lthreads = malloc(sizeof(pthread_t) * clnt);
- /* initialisation de la communication avec le maitre */
/* Initialize client to master pipe */
if (pipe(dp.maitre) < 0) {
perror("Master pipe creation error\n");
@@ -285,7 +264,6 @@ void lockWholeFile(struct flock *request)
{
request->l_whence = SEEK_SET;
request->l_start = 0;
- /* Lock de l'ensemble du fichier */
/* Lock the whole file */
request->l_len = 0;
}
@@ -319,10 +297,7 @@ void rapport(clnt)
int serverSendLocal()
{
int i;
- /* Synchronisation des processus esclaves. */
/* Synchronize slave processes */
-
- /* On configure les esclaves pour le test */
/* Configure slaves for test */
for (i = 0; i < dp.nclnt; i++)
@@ -479,7 +454,6 @@ int getResults(int ntest)
{
int i, c;
int resultat = 0;
- /* On comptabilise les resultats distants */
/* Add remote test results */
for (c = 0; c < maxClients; c++) {
for (i = 0; i < dp.nclnt; i++) {
@@ -489,7 +463,6 @@ int getResults(int ntest)
}
}
- /* On comptabilise les resultats locaux */
/* Add local test results */
for (i = 0; i < dp.nclnt; i++) {
read(maitreLecteur, message, M_SIZE);
@@ -500,18 +473,13 @@ int getResults(int ntest)
return 0;
}
-/* Usefull debug macro */
#ifdef DEBUG
#define P(a,b) memset(dbg,0,16);sprintf(dbg,a,b);write(0,dbg,16);
#endif
-/* Le maitre de l'application client n'est qu'un repetiteur d'information.
- * Il retransmet l'information qu'il reçoit du maitre du client a ses esclaves
- */
/* In the case of a network use, the master of the client application si only
* a 'repeater' of information. It resends server-master instructions to its own slaves.
*/
-
void maitreClient()
{
fd_set fdread;
@@ -530,7 +498,6 @@ void maitreClient()
n = fdServeur > maitreLecteur ? fdServeur : maitreLecteur;
printf("Maitre CLient - fdServeur=%d\n", fdServeur);
while (1) {
- /* On ajoute les descripteurs esclave et serveur */
/* Add slave and server pipe file descriptors */
FD_ZERO(&fdread);
FD_SET(fdServeur, &fdread);
@@ -545,9 +512,6 @@ void maitreClient()
}
if (FD_ISSET(fdServeur, &fdread)) {
- /* On vient de recevoir une information du serveur
- * On la retransmet aux esclaves
- */
/* We just have received information from the server.
* We repeat it to slaves.
*/
@@ -555,9 +519,7 @@ void maitreClient()
t = message[0];
switch (t) {
case 'L':
- /* Instruction : Lock. Dans ce cas il faut envoyer a chaque processus une section differente a locker */
/* Lock instruction. We need to send a different section to lock to each process */
-
unSerialiseFLock(&lock);
start = lock.l_start;
for (i = 0; i < dp.nclnt; i++) {
@@ -568,12 +530,7 @@ void maitreClient()
printf("\n");
continue;
case 'T':
- /* Instruction: Test. Il s'agit d'une trame annoncant un nouveau test : on verifie qu'il ne s'agit pas d'une
- * demande de FIN des tests
- */
/* Test instruction. Ensure server is not sending the FIN(ish) instruction to end tests */
-
- /* A re-ecrire un peu mieux des que possible */
/* To be rewritten asap */
m = atoi(&(message[2]));
if (m == FIN)
@@ -586,8 +543,6 @@ void maitreClient()
}
break;
} else {
- /* Dans le cas inverse, on lis les esclaves et on remonte l'information au serveur
- */
/* Else, we read information from slaves and repeat them to the server */
for (i = 0; i < dp.nclnt; i++) {
r = read(maitreLecteur, message, M_SIZE);
@@ -600,15 +555,12 @@ void maitreClient()
}
}
- /* On vient de recevoir la trame FIN de programme */
/* Receive the FIN(ish) instruction */
- /* On la communique a tous les esclaves */
/* Repeat it to the slaves */
printf("Fin du programme en cours...\n");
serverSendLocal();
- /* Et c'est fini! */
/* Ok, we can quit */
printf("Bye :)\n");
@@ -633,14 +585,12 @@ void maitre()
clnt = dp.nclnt;
maitreLecteur = dp.maitre[0];
etat = SELECT;
- /* On commence par le premier test. C'est original ;) */
/* Start with the first test ;) */
n = 0;
printf("\n--------------------------------------\n");
while (1) {
switch (etat) {
case SELECT:
- /* Selection du test a effectuer */
/* Select the test to perform */
printf("\n");
E("Maitre: SELECT");
@@ -674,7 +624,6 @@ void maitre()
continue;
case LOCK:
- /* On applique le lock que l'on veut */
/* Apply the wanted lock */
E("Maitre: LOCK");
write(dp.fd, phraseTest, strlen(phraseTest));
@@ -701,15 +650,6 @@ void maitre()
continue;
case BYTELOCK:
- /*
- * L'idee est de faire locker l'ensemble du fichier octet par octet par un ensemble de sous processus
- * Il nous faut donc
- * -creer un fichier ayant autant d'octet que le nombre de processus passe en parametres
- * -passer les sections a locker a chacun des esclaves
- * -verifier que les locks sont bien appliques
- *
- */
-
/* The main idea is to lock all the bytes in a file. Each slave process locks one byte.
*
* We need :
@@ -718,9 +658,6 @@ void maitre()
* - ensure locks have been correctly set
*/
- /* On cree une chaine de caracteres a enregistrer dans le fichier qui contienne exactement un octet par
- * processus.
- */
/* Create a string to record in the test file. Length is exactly the number of sub process */
P("Maitre: BYTELOCK: %d\n", etat);
buf = malloc(clnt * (maxClients + 1));
@@ -728,24 +665,19 @@ void maitre()
write(dp.fd, buf, clnt);
free(buf);
- /* Chaque processus esclave reecrit son champs a locker. */
/* Each slave process re-writes its own field to lock */
request.l_whence = SEEK_SET;
request.l_start = 0;
request.l_len = 1;
- /* On commence par les envois reseau */
/* Start to send sections to lock to remote process (network clients) */
-
for (i = 0; i < maxClients; i++) {
- /* On renseigne la structure avec le lock qui va bien */
/* Set the correct byte to lock */
offset = (i + 1) * clnt;
request.l_start = (off_t) offset;
serverSendLockClient(&request, i);
}
- /* Puis les envois locaux */
/* Now send sections to local processes */
for (i = 0; i < clnt; i++) {
request.l_start = i;
@@ -768,7 +700,6 @@ void maitre()
continue;
case RESULTAT:
- /* On lit les resultats un par un */
/* Read results by one */
getResults(n - 1);
if (bl)
@@ -777,15 +708,12 @@ void maitre()
continue;
case CLEAN:
- /* On demande aux clients d'arreter le test */
/* Ask the clients to stop testing ... */
tLock.test = CLEAN;
serialiseTLock(&tLock);
serverSend();
- /* ... et on attend un accuse de reception avant de fermer */
/* ... and wait for an ack before closing */
serverReceive();
- /* On ignore les resultats, ce n'est qu'un accuse de reception */
/* Ignore message content : that is only an ack */
/* close and open file */
@@ -803,9 +731,9 @@ void maitre()
printf("(end)\n");
exit(0);
- } /* switch */
+ }
break;
- } /* while */
+ }
rapport(clnt);
}
@@ -847,7 +775,6 @@ void *esclave(void *donnees)
continue;
case RDONLY:
- /* On tente de lire un fichier */
/* Try to read a file */
P("TEST READ ONLY %d\n", RDONLY);
if ((ftest = open(dp.fname, O_RDONLY | O_NONBLOCK)) < 0) {
@@ -867,7 +794,6 @@ void *esclave(void *donnees)
continue;
case WRONLY:
- /* On tente d'ecrire un fichier */
/* Try to write a file */
P("TEST WRITE ONLY %d\n", WRONLY);
if ((ftest = open(dp.fname, O_WRONLY | O_NONBLOCK)) < 0) {
@@ -888,7 +814,6 @@ void *esclave(void *donnees)
case LOCK:
case READLOCK:
- /* On essaie de lire une section lockee en lecture sur le fichier */
/* Try to read a read-locked section */
P("READ LOCK %d\n", F_RDLCK);
if ((ftest = open(dp.fname, O_RDONLY | O_NONBLOCK)) < 0) {
@@ -900,7 +825,6 @@ void *esclave(void *donnees)
}
P("fd=%d\n", ftest);
- /* Lock de l'ensemble du fichier */
/* Lock the whole file */
request.l_type = F_RDLCK;
request.l_whence = SEEK_SET;
@@ -917,7 +841,6 @@ void *esclave(void *donnees)
continue;
case WRITELOCK:
- /* On essaie d'ecrire le fichier */
/* Try to write a file */
P("WRITE LOCK %d\n", F_WRLCK);
if ((ftest = open(dp.fname, O_WRONLY | O_NONBLOCK)) < 0) {
@@ -927,7 +850,6 @@ void *esclave(void *donnees)
perror("\nOpen:");
continue;
}
- /* Lock de l'ensemble du fichier */
/* Lock the whole file */
P("fd=%d\n", ftest);
request.l_type = F_WRLCK;
@@ -955,11 +877,7 @@ void *esclave(void *donnees)
continue;
case BYTELOCK:
- /* On se met en attente de la section a locker. L'ensemble de la structure est
- * donnee par le maitre et transmise par le tube.
- */
/* Wait for the exact section to lock. The whole file is sent by the master */
-
P("BYTE LOCK %d\n", etat);
getLockSection(&request);
if ((ftest = open(dp.fname, O_RDWR | O_NONBLOCK)) < 0) {
@@ -977,7 +895,6 @@ void *esclave(void *donnees)
etat = RESULTAT;
continue;
}
- /* On change le caractere a la place donnee pour verification */
/* Change the character at the given position for an easier verification */
a = lseek(ftest, request.l_start, SEEK_SET);
write(ftest, "=", 1);
@@ -1077,7 +994,6 @@ int main(int argc, char **argv)
serveur = 0;
continue;
}
- /* Option inconnue */
printf("Ignoring unknown option: %s\n", argv[i]);
}
diff --git a/testcases/network/nfsv4/locks/locktests.h b/testcases/network/nfsv4/locks/locktests.h
index 34ba36461..eba856a0e 100644
--- a/testcases/network/nfsv4/locks/locktests.h
+++ b/testcases/network/nfsv4/locks/locktests.h
@@ -90,7 +90,7 @@
#define M_SIZE 512
-int writeToAllClients(char *foo);//#define DEBUG
+int writeToAllClients(char *foo);
int serverReceiveNet();
int clientReceiveNet();
@@ -118,36 +118,26 @@ enum etat_t {
BYTELOCK_WRITE
};
-/* Donnees communes aĆ tous les processu */
/* Public data */
struct donneesPub {
- /* Nombre de clients */
/* Number of clients */
int nclnt;
- /* Liste des clients (liste des tubes)*/
/* List of master to slave pipes */
int **lclnt;
- /* Tube de communication avec le maitre */
/* Slave to master pipe */
int maitre[2];
- /* Liste des threads */
/* Thread list */
pthread_t *lthreads;
- /* nom du fichier test */
/* test file name */
char *fname;
- /* descripteur du fichier test */
/* test file file-descriptor */
int fd;
- /* Affichage des messages d'erreur */
/* Detailed error messages */
int verbose;
};
-/* Donnees privees aux processus */
/* private data */
struct donneesPriv {
- /* Numero de thread. */
/* thread number */
int whoami;
};
diff --git a/testcases/network/nfsv4/locks/netsync.c b/testcases/network/nfsv4/locks/netsync.c
index be0d04398..34dabc9bd 100644
--- a/testcases/network/nfsv4/locks/netsync.c
+++ b/testcases/network/nfsv4/locks/netsync.c
@@ -15,7 +15,6 @@ int serverReceiveClient(int c)
{
char tmp[M_SIZE];
int r, s;
- /* Il faut etre sur que l'on lit _exactement_ la trame envoyee (M_SIZE) */
/* Ensure we read _exactly_ M_SIZE characters in the message */
memset(message, 0, M_SIZE);
memset(tmp, 0, M_SIZE);
@@ -24,8 +23,7 @@ int serverReceiveClient(int c)
while (s < M_SIZE) {
r = read(fdClient[c], tmp, M_SIZE - s);
- /* On complete le message au fur et a mesure */
- /* Loop until we have a complete message */
+ /* Loop until we have a complete message */
strncpy(message + s, tmp, r);
s += r;
}
@@ -73,9 +71,6 @@ int setupConnectionServeur()
}
size = sizeof(struct sockaddr_in);
for (c = 0; c < maxClients; c++) {
-
- /* On accepte les connections clientes */
- /* Accept incoming connections */
if ((fdClient[c] =
accept(sock, (struct sockaddr *)&remote, &size)) == -1) {
perror("accept");
@@ -111,15 +106,6 @@ int writeToAllClients(char *foo)
int setupClients(int type, char *fname, int nThread)
{
/*
- * Envoi des parametres a tous les clients
- *
- * on doit envoyer 3 parametres :
- * - l'emplacement du fichier test
- * - Le nombre d'esclaves
- * - Le type de sous processus : thread ou process
- */
-
- /*
* Send parameters to all slaves :
*
* We must send :
@@ -127,7 +113,6 @@ int setupClients(int type, char *fname, int nThread)
* - the number of slaves for each client
* - The kind of slaves : process or thread
*/
-
char message[512];
sprintf(message, "%d:%s:%d::", type, fname, nThread);
writeToAllClients(message);
@@ -176,7 +161,6 @@ int readFromServer(char *message)
{
char tmp[M_SIZE];
int r, s;
- /* Il faut etre sur que l'on lit _exactement_ la trame envoyee de taille M_SIZE */
/* Ensure we read exactly M_SIZE characters */
memset(message, 0, M_SIZE);
memset(tmp, 0, M_SIZE);
@@ -184,7 +168,6 @@ int readFromServer(char *message)
s = 0;
while (s < M_SIZE) {
r = read(fdServeur, tmp, M_SIZE - s);
- /* On complete le message au fur et a mesure */
/* Loop until we have a complete message */
strncpy(message + s, tmp, r);
s += r;
--
2.13.0
More information about the ltp
mailing list