Discussion :

[rufa11]

Bonjour,
voila mon problèmes et le suivant:
je doit comparait entre deux chaine de carctères pour dans une boucle,masi je n'arrive pas a trouver la solution, le code source et le suisvant:

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
static int bond_xmit_nouveau(struct sk_buff *skb, struct net_device *bond_dev)
{
       struct bonding *bond = netdev_priv(bond_dev);
       struct slave *slave;
       int i;
       read_lock(&bond->lock);

       if (!BOND_IS_OK(bond)) {
               goto out;
       }

       bond_for_each_slave(bond, slave, i) {
       if( *(slave->dev->name) == 'eth0' )    
       printk("\n l'interface est: %s \n", slave->dev->name);                       
       }
out:
      read_unlock(&bond->lock);
      return 0; 
}

où

char name[IFNAMSIZ]; /* name contient le nom de l'interface */
#define IFNAMSIZ 16
slave: et une structure qui contien un champ de type poiteure dev.
dev: est un poiteure vers une structure de type net_device .
net_device: est une structure qui contient le champ name déclaré si dessous.

l'erreur:

Code :

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

1
2
3
4
5
6
7
8
9
10
root@abdelkader-desktop:/usr/src/linux-2.6.29# ./do
compilation
  CC [M]  drivers/net/bonding/bond_main.o
drivers/net/bonding/bond_main.c:4373:35: warning: multi-character character constant
 LD [M]  drivers/net/bonding/bonding.o
  Building modules, stage 2.
  MODPOST 1 modules
  CC      drivers/net/bonding/bonding.mod.o
  LD [M]  drivers/net/bonding/bonding.ko
installation des mises à jours

Merci d'avance.

[kamouminator]

oulla...
d'abord il faut baliser ton code pour que ce soit lisible...
ensuite, on ne compare pas comme ca deux chaine de caractere !

pour comparer deux chaine: strcmp(const char *, const char *)
strcmp renvoi 0 si les chaines sont egales


et 'eth0' ne veut absolument rien dire...si tu veux ecrire une chaine c'es les doubles quotes qu'il faut utiliser: "eth0"

ecrire un driver avec de telles erreures peut être fatal pour le noyau.
tu devrai revoir les bases du C avant de te lancer dans le developpement d'un module.

[Sve@r]

Je présume que ton code est du C (ou C++). Ben le problème c'est que dans ce langage bas niveau, le type "chaine" n'existe pas. Tout ce que le C connait, ce sont "les types simples (char, int, short, float, double)", "les tableaux" et "les pointeurs (adresses de variables)"
Et donc pour pouvoir traiter une chaine, il faut la stocker dans un tableau de char.
Exemple

Code :

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

char chaine[]="Bonjour";

Cependant, malgré cette déclaration, tout ce que tu as en main, ce n'est que l'adresse du premier caractère (l'adresse de la case contenant 'B'). Avec cette adresse tu sais que sur la case suivante, tu auras 'o' puis 'n' etc.

Donc si tu veux comparer 2 chaines, il te faut boucler sur chaque caractère de la chaine 1 en parallèle avec chaque caractère de la chaine 2 et comparer les caractères un à un.
Heureusement cet algo a déjà été implémenté par les programmeurs de la librairie et codé dans la fonction strcmp(). Cette fonction reçoit les adresses de début de chaine, s'occupe elle-même de la boucle et renvoie 0 si les chaines sont égales, -1 si la chaine 1 est placée avant la chaine 2 dans l'ordre alphabétique et +1 si c'est l'inverse.

Code :

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

1
2
3
4
if (strcmp(chaine, "Bonjour") == 0)
    printf("Chaines égales\n");
else
    printf("Chaines différentes\n");

Remarque personnelle: vaut mieux apprendre les bases du C et faire des exercices avant de commencer à coder un truc ressemblant déjà à un vrai projet. Par ailleurs, quand je vois du "goto" j'ai un certain "a priori négatif" sur le codeur...

[rufa11]

Bonjour, oui strcmp a résolu le problème, je les deja vu dans string.c mais je les complètement oublier, je débute encore dans la programmation kernel, c'est difficile mais tres intéréssent.
Pour le "goto" je n'est fait que suivre le stile de programmation adopté par le programmeur du module bonding,j'ai pas voulu changer de stile.
Merci milles foit, vous m'avait fait gagner beacoup de temps.

[Sve@r]

je débute encore dans la programmation kernel, c'est difficile mais tres intéréssent.

Whaou. Pour un débutant C, tu te lances de suite dans la haute voltige !!!

Pour le "goto" je n'est fait que suivre le stile de programmation adopté par le programmeur du module bonding,j'ai pas voulu changer de stile.

C'est pas forcément négatif... mais avant de le mettre faut quand-même se demander s'il n'y a pas une autre solution.

[kamouminator]

dans ton cas le goto ne sert à rien,
le goto est generalement utilisé dans les modules lorsqu'une suite d'allocation est realisée pour eviter de reecrire 36000 fois la meme chose:

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
 
 
if(!alloc1())
  goto err1;
if(!alloc2())
  goto err2
if(!alloc3())
  goto err3
 
/*...*/
err3:
free2();
err2:
free1();
err1:
 
return -ERRNO;

[rufa11]

re, oui je voi ce que vous voulais dire, mai comment éviter le bloc dans le ca ou le bond n'est pas pres a la tx si je n'utilise pas le goto? il le faut que le teste soit en premier lieu. Et la fonction n'est pas encore terminée,il reste encore le marquage des paquets ip et le arp monitoring.
Merci.

[kamouminator]

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
 
 
static int bond_xmit_nouveau(struct sk_buff *skb, struct net_device *bond_dev)
{
  struct bonding *bond = netdev_priv(bond_dev);
  struct slave *slave;
  int i;
  read_lock(&bond->lock);
 
  if (BOND_IS_OK(bond)) {
    bond_for_each_slave(bond, slave, i) {
    if( !strcmp(slave->dev->name ,"eth0" ) 
      printk("\n l'interface est: %s \n", slave->dev->name); 
    }
  }
  read_unlock(&bond->lock);
  return 0; 
}

par exemple...

[rufa11]

Bonjour, oui ça a l'aire beaucoup plus optimisant, je vai tenire compte.
un grand Merci.

[kamouminator]

je n'ai fais que copier ton code et changer le if,
il reste une grosse erreur dans ton code, je ne sais pas ce que fais exactement bond_for_each_slave mais le pointeur slave n'a pas été alloué !

[rufa11]

Re, vos conseilles met très précieux Mr.oui moi aussi au début je pensé que make va me le signaler,mais elle ne me la pas signaler,que "slave déclared but uninitialized". je me suis inspiré de :

Code :

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

1
2
3
4
5
6
7
8
9
 struct slave *slave;
int slave_no;   
 slave_no = bond->xmit_hash_policy(skb, bond_dev, bond->slave_cnt);
bond_for_each_slave(bond, slave, i) 
                slave_no--;
                if (slave_no < 0) {
                        break;
                }
        }

dans ce ca aussi il na pas initialisé la varible slave et pourtant quand je compile le module il ne me signale pas que slave non intialisé. parcontre si je fait:

Code :

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

1
2
struct slave   *slave;
printk("\n le nom de l'interface est: %s", slave->dev->name);

il m'affiche un warning et si je lance "ifenslave bond0 ethx" alors la c'est tous le système qui va se planté,et c'est normale vu qu'il n y a rien a afficher.

La boucle bond_for_each_slave sert a itérer la liste des slave.

[kamouminator]

en fait, bond_for_each_slave est une macro, en regardant dans les source j'ai trouvé que slave était initialisé de la sorte:

Code :

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

slave = (bond)->first_slave

[rufa11]

oui c'est vrai, les pointeurs comme bond->curr_active_slave et bond->first_active_slave sont initialisés dans bond_init().