RE: [Tsp-devel] patch RPC cherche testeurs sous Solaris

[Eric.NOULARD]

Je pense pmpat_unset est insuffisant
il faut au moins de-enregistrer le service

svc_unregister
+
pmap_unset

le problème c'est même ça fait [apparemment]
pas sortir svc_run
pour que tu évites de chercher j'ai aussi testé

un pthread_cancel sur le thread qu'on a lancé
et qui exécute svc_run.

Ca à l'air de marcher pas terrible, j'ai aussi un
core dump mais avec une tronche différente.

Je suis pas arrivé à debugger car ça se termine
dans la glibc dont je n'aivait pas de version de
debug avec laquelle me linker.
...

En tous les cas si on a un souci multi-plateforme
be on peut aussi régler ça par
un

#ifdef DEC_OSF
ou
#ifdef LINUX

etc...

Que la force soit avec toi Bob :))
Eric
-----Original Message-----
From:   address@hidden on behalf of PAGNOT, Robert
Sent:   Thu 10/28/2004 8:39 AM
To:     tsp-devel
Cc:    
Subject:        RE: [Tsp-devel] patch RPC cherche testeurs sous Solaris
Gloups .....

J'avais commencé avec un svc_exit, mais cette fonction n'existe pas sous
DEC/OSF, alors j'étais dans la m....

Le problème consiste à forcer un return svc_run. Alors comment faire ?

Mon idée était de supprimer le mapping port/service (pmap_unset) pour
débloquer le select de svc_run (réponse à la question 1) de Erk). Puis de
libérer ce qui avait été alloué.

Le code de la 1.18 marche sur toutes les combinaisons que j'ai : Linux 2.4,
Solaris 2.5/2.8 et DEC/OSF 5.1.
Depuis peu je dispose d'un Linux 2.6, mais je n'ai vraiment pas eu le temps
de tester tout cela.

Je prends le patch, et j'essaye voir ce que cela donne ...

A+


Robert PAGNOT
ASTRIUM SAS
AEA56 - Ground Systems
tél.: 05 62 19 55 32
fax.: 05 62 19 77 41
<mailto:address@hidden>



> -----Original Message-----
> From: Stephane Galles [mailto:address@hidden]
> Sent: Thursday, October 28, 2004 1:00 AM
> To: tsp-devel
> Subject: Re: [Tsp-devel] patch RPC cherche testeurs sous Solaris
>
>
>
> Oublie mon histoire de test sous Linux, on dirait bien que
> c'est nornal
> que cela plante.
>
> Au niveau de la ligne de code dont tu parles on trouve :
>
>  STRACE_DEBUG(("calling svc_exit..."));
>  svc_destroy(config->xprt);
>
> Le probleme est qu'on ne stoppe pas le svc_run, il faudrait :
>
> STRACE_DEBUG(("calling svc_exit..."));
> svc_exit();   /* svc_run s'arrete */
> svc_destroy(config->xprt);
>
> (encore que je ne sais plus si le svc_destroy est necessaire
> apres un exit)
>
> J'ai vérifié dans CVS, le svc_exit était là jusqu'a la version 1.17,
> puis il a ete remplacé par le svc_destroy dans la 1.18. Mais comme je
> n'ai pas tout suivi du developpement de la partie C, je vous
> laisse vous
> arranger entre vous  ;)
>
>
> A+
>
> Steph
>
>
> NOULARD Eric wrote:
>
> >Après quelques investigations il s'avère que
> >c'est la fonction svc_destroy
> >qui semble planter sous Linux (noyau 2.6.8.1 glibc 2.3.3)
> >si on l'enlève tout bonnement et qu'on fait
> >seulement
> >
> >svc_unregister
> >pmap_unset
> >
> >ben c'est ok??
> >
> >D'où 2 questions:
> >
> >1) A quoi sert réellement svc_destroy?
> >   l'argument de type SVCXPRT* qui est détruit par svc_destroy
> >   est utilise (probablement) par svn_run, suite au svc_tcpcreate
> >   +svc_register. Hors svc_run ne revient jamais, donc que
> se passe-t-il
> >   quand on "svc_destroy" quand svc_run n'est pas terminé?
> >
> >2) est-ce que le patch qui suit serait OK sous Solaris
> >  Patch applicable avec un
> >  cd $DEVBASE
> >  patch -p0 < rpc_linux.patch
> >
> >Des volontaires pour tester ça sous Solaris
> >avant que je commite ces changements?
> >
> >A noter que j'ai donné la version de la glibc car sous Linux
> >la lib [sun]rpc fait visiblement de la glibc
> >dans les sources (par exemple de la 2.3.3)
> >glibc-2.3.3/sunrpc/rpc/svc.h:
> >
> >#define svc_destroy(xprt)                               \
> >        (*(xprt)->xp_ops->xp_destroy)(xprt)
> >
> >Ce qui ne m'aide pas beaucoup...
> >
> > 
> >
> >-------------------------------------------------------------
> -----------
> >
> >Index: src/core/rpc/tsp_server.c
> >===================================================================
> >RCS file: /cvsroot/tsp/tsp/src/core/rpc/tsp_server.c,v
> >retrieving revision 1.18
> >diff -r1.18 tsp_server.c
> >39a40
> > 
> >
> >>/* FIXME a quoi sert ce define ? */
> >>   
> >>
> >42a44,46
> > 
> >
> >>#include <rpc/pmap_clnt.h>
> >>#include <stdio.h>
> >>#include <unistd.h>
> >>   
> >>
> >45a50
> > 
> >
> >>#include "../ctrl/tsp_provider.h"
> >>   
> >>
> >50d54
> ><
> >51a56,60
> > 
> >
> >>#include "tsp_rpc_confprogid.h"
> >>
> >>void
> >>tsp_rpc_1(struct svc_req *rqstp, register SVCXPRT *transp) ;
> >>
> >>   
> >>
> >54a64
> > 
> >
> >>#define TSP_URL_MAXLENGTH 256
> >>   
> >>
> >58c68
> ><   char url[256];
> >---
> > 
> >
> >>  char url[TSP_URL_MAXLENGTH];
> >>   
> >>
> >64,71c74,76
> ><  
> ><   static TSP_provider_info_t server_info;
> ><   
> ><   STRACE_IO(("-->IN"));
> ><
> ><    
> ><   server_info.info = GLU_get_server_name();
> ><    
> >---
> > 
> >
> >>  static TSP_provider_info_t server_info;  
> >>  STRACE_IO(("-->IN"));   
> >>  server_info.info = GLU_get_server_name();   
> >>   
> >>
> >73,75d77
> ><
> ><   
> ><
> >84,93c86,88
> ><
> ><
> ><   STRACE_IO(("-->IN"));
> ><
> ><   
> ><   TSP_provider_request_open(&req_open, &ans_open);
> ><   
> ><   STRACE_IO(("-->OUT"));
> ><
> ><   
> >---
> > 
> >
> >>  STRACE_IO(("-->IN"));    
> >>  TSP_provider_request_open(&req_open, &ans_open); 
> >>  STRACE_IO(("-->OUT"));   
> >>   
> >>
> >117,119c112
> ><   STRACE_IO(("-->IN"));
> ><
> ><   
> >---
> > 
> >
> >>  STRACE_IO(("-->IN"));    
> >>   
> >>
> >121,124c114
> ><
> ><   STRACE_IO(("-->OUT"));
> ><
> ><   
> >---
> > 
> >
> >>  STRACE_IO(("-->OUT"));      
> >>   
> >>
> >135d124
> ><
> >137d125
> ><
> >150d137
> ><
> >160d146
> ><
> >173d158
> ><
> >180d164
> ><
> >183,186d166
> ><
> ><   
> ><    
> ><
> >189d168
> ><   
> >200d178
> ><
> >202d179
> ><
> >221,224d197
> ><
> >< void
> >< tsp_rpc_1(struct svc_req *rqstp, register SVCXPRT *transp) ;
> ><
> >260,261c233,234
> ><    /* svc_create does not exist for linux, we must use the
> deprecated function */
> ><   pmap_unset (rpc_progid, TSP_RPC_VERSION_INITIAL);
> >---
> > 
> >
> >>  /* svc_create does not exist for linux, we must use the
> deprecated function */
> >>  //pmap_unset (rpc_progid, TSP_RPC_VERSION_INITIAL);
> >>   
> >>
> >285a259,261
> > 
> >
> >>  //STRACE_DEBUG(("calling svc_exit..."));
> >>  //svc_destroy(config->xprt);
> >>
> >>   
> >>
> >288a265
> > 
> >
> >>     
> svc_unregister(TSP_get_progid(config->server_number),TSP_RPC_V
> ERSION_INITIAL);
> >>   
> >>
> >291,293d267
> ><
> ><   STRACE_DEBUG(("calling svc_exit..."));
> ><   svc_destroy(config->xprt);
> >314c288
> ><   TSP_rpc_request_config_t *config =
> (TSP_rpc_request_config_t *)this->config_param;
> >---
> > 
> >
> >>  TSP_rpc_request_config_t *config =
> (TSP_rpc_request_config_t *)(this->config_param);
> >>   
> >>
> >318,319c292,294
> ><   strcpy(config->url, "");
> ><
> >---
> > 
> >
> >>  /* do not buffer overflow */
> >>  memset(&config->url[0],'\0',TSP_URL_MAXLENGTH);
> >> 
> >>   
> >>
> >332c307,308
> ><       sprintf(config->url, TSP_URL_FORMAT,
> TSP_RPC_PROTOCOL, hostname, servername, config->server_number);
> >---
> > 
> >
> >>      snprintf(config->url, TSP_URL_MAXLENGTH,
> >>           TSP_URL_FORMAT, TSP_RPC_PROTOCOL, hostname,
> servername, config->server_number);
> >>   
> >>
> >346c322
> ><   TSP_rpc_request_config_t *config =
> (TSP_rpc_request_config_t *)this->config_param;
> >---
> > 
> >
> >>  TSP_rpc_request_config_t *config =
> (TSP_rpc_request_config_t *)(this->config_param);
> >>   
> >>
> >350c326
> ><   pthread_detach(pthread_self()); /* FIXME shoudl we do this */
> >---
> > 
> >
> >>  //pthread_detach(pthread_self()); /* FIXME shoudl we do this */
> >>   
> >>
> >366c342
> ><   TSP_rpc_request_config_t *config =
> (TSP_rpc_request_config_t*)this->config_param;
> >---
> > 
> >
> >>  TSP_rpc_request_config_t *config =
> (TSP_rpc_request_config_t*)(this->config_param);
> >>   
> >>
> >375c351
> ><   TSP_rpc_request_config_t *config =
> (TSP_rpc_request_config_t*)this->config_param;
> >---
> > 
> >
> >>  TSP_rpc_request_config_t *config =
> (TSP_rpc_request_config_t*)(this->config_param);
> >>   
> >>
> >385c361
> >< void main(void)
> >---
> > 
> >
> >>int main(void)
> >>   
> >>
> >392a369
> > 
> >
> >>  return 0;
> >>   
> >>
> >>------------------------------------------------------------
> ------------
> >>
> >>_______________________________________________
> >>Tsp-devel mailing list
> >>address@hidden
> >>http://lists.nongnu.org/mailman/listinfo/tsp-devel
> >>   
> >>
>
>
>
> _______________________________________________
> Tsp-devel mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/tsp-devel
>