Discussion :

[Cédric-29]

Bonjour,
les macros pour la déclaration des workqueue ont récemment changée. Auparavant, on avait des macros du type :

Code :

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

1
2
3
4
 
DECLARE_WORK(name, void (*function)(void *), void *data);
INIT_WORK(struct work_struct *work, void (*function)(void *), void *data);
PREPARE_WORK(struct work_struct *work, void (*function)(void *), void *data);

pour déclarer et préparer une tâche en l'associant à une fonction, function, et indiquer les paramètres d'entrée, data, de cette fonction. Depuis le noyau 2.6.20 (je crois), les macros ont été revues :

Code :

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

1
2
3
4
 
DECLARE_WORK(name, void (*function)(void *));
INIT_WORK(struct work_struct *work, void (*function)(void *));
PREPARE_WORK(struct work_struct *work, void (*function)(void *));

Quelqu'un sait-il comment l'on doit faire pour indiquer les paramètres à passer à la fonction ?

Merci d'avance,
Cédric

[nicolas.sitbon]

Effectivement et à l'habitude des développeurs kernel linux, je n'ai pas réussi à avoir plus d'explication.
On avait en 2.6.19.7:

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
/*
  * initialize a work-struct's func and data pointers:
  */
 #define PREPARE_WORK(_work, _func, _data)                       \
         do {                                                    \
                 (_work)->func = _func;                          \
                 (_work)->data = _data;                          \
         } while (0)
  /*
   * initialize all of a work-struct:
   */
#define INIT_WORK(_work, _func, _data)                          \
         do {                                                    \
                 INIT_LIST_HEAD(&(_work)->entry);                \
                 (_work)->pending = 0;                           \
                 PREPARE_WORK((_work), (_func), (_data));        \
                 init_timer(&(_work)->timer);                    \
         } while (0)

puis en 2.6.20:

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
#define PREPARE_WORK(_work, _func)                              \
        do {                                                    \
                (_work)->func = (_func);                        \
        } while (0)
 
/*
 * initialize all of a work item in one go
 *
 * NOTE! No point in using "atomic_long_set()": useing a direct
 * assignment of the work data initializer allows the compiler
 * to generate better code.
 */
#define INIT_WORK(_work, _func)                                 \
        do {                                                    \
                (_work)->data = (atomic_long_t) WORK_DATA_INIT(0);      \
                INIT_LIST_HEAD(&(_work)->entry);                \
                PREPARE_WORK((_work), (_func));                 \
        } while (0)

Je ne suis pas certain que tu puisses encore passer un argument...

[jowo]

Bonjour,

Je n'ai de connaissance particulière sur le kernel mais en comparant les deux versions, on remarque que les données sont initialisées durant la macro INIT_WORK

Code :

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

1
2
 
(_work)->data = (atomic_long_t) WORK_DATA_INIT(0);

Mais je ne sais où la macor WORK_DATA_INIT est définie.

[nicolas.sitbon]

Bonjour,

Je n'ai de connaissance particulière sur le kernel mais en comparant les deux versions, on remarque que les données sont initialisées durant la macro INIT_WORK

Code :

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

1
2
 
(_work)->data = (atomic_long_t) WORK_DATA_INIT(0);

Mais je ne sais où la macor WORK_DATA_INIT est définie.

J'avais déjà regardé mais à priori ça ne t'avancera à rien:

Code :

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

1
2
#define WORK_DATA_INIT(autorelease) \
        ATOMIC_LONG_INIT((autorelease) << WORK_STRUCT_NOAUTOREL)

Code :

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

#define WORK_STRUCT_NOAUTOREL 1         /* F if work item automatically released on exec */

Code :

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

#define ATOMIC_LONG_INIT(i)     ATOMIC_INIT(i)

Code :

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

#define ATOMIC_INIT(i)  { (i) }

[Cédric-29]

Merci de votre intérêt pour ce problème. C'est tout de même étrange que ces modifs soient si peu documentées.
j'ai vu que certaines personnes arrivaient à se débrouiller en utilisant la macro container_of pour passer les données à la fonction. Ca me parait être de la bidouille.

Je vais finir par utiliser les Tasklets, elle ont encore une macro a trois arguments permettant de passer des données. Mais pour combien de temps ? ...

[nicolas.sitbon]

Merci de votre intérêt pour ce problème. C'est tout de même étrange que ces modifs soient si peu documentées.
j'ai vu que certaines personnes arrivaient à se débrouiller en utilisant la macro container_of pour passer les données à la fonction. Ca me parait être de la bidouille.

Je vais finir par utiliser les Tasklets, elle ont encore une macro a trois arguments permettant de passer des données. Mais pour combien de temps ? ...

C'est un vrai problème je te l'accorde, voici un problème auquel j'ai été confronté avec sys_futex():

since FUTEX_FD was scheduled for removal in June 2007 lets remove it.
Google Code search found no users for it and NGPT was abandoned in 2003
according to IBM. futex.h is left untouched to make sure the id does
not get reassigned. Since queue_me() has no users left it is commented
out to avoid a warning, i didnt remove it completely since it is part
of the internal api (matching unqueue_me())

Le monde du libre et en particuliers les développeurs kernel ont un gros travail à faire sur eux même.
Bon courage.