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

[Eric.NOULARD]

Ben voilà une nouvelle qu'elle est bonne
c'est beau la puissance de la mémoire collective :))

Je sais pas qui a enlevé le svc_exit()
mais il est bien possible qu'il paie
un Caribou arrosé d'un seau de bière pour ça!!
sans dénoncer les amis je lis dans les log CVS:

1.17--> 1.18
2 weeks 
tractobob       
Try svc_destroy instead of svc_exit that does not exist on DEC

ben est-ce que ce serait pas DEC qui fait chier ?!??

Plus sérieusement, si c'est une histoire d'obsolescence de l'API
sun rpc j'aimerais qu'on nous le dise !!

Quoiqu'il en soit j'ai trouvé pas mal d'info
sur les RPC mais RIEN sur une quelconque
'normalisation de l'API'

la couche réseau c'est sans souci 'ONC RPC' RFC
RFC1831 et RFC1832 (cf www.rfc-editor.org)

mais l'API ben y'a guère que dans le code
source de la glibc que je l'ai trouvé...

ou dans le man rpc (sous linux) qui ne parle
en aucun cas de svc_exit :))


Eric

-----Original Message-----
From:   address@hidden on behalf of Stephane Galles
Sent:   Thu 10/28/2004 1:00 AM
To:     tsp-devel
Cc:    
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_VERSION_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