Re: [BProc] re: bpsh hangs

[Nicholas H. <he...@se...>]

Yup -- didnt even notice that, but those are the pids that were hanging.
I will try the patch as well -- it should only take a day to tell if it
works :-)

Nic

On Fri, 18 Jan 2002, Erik Arjan Hendriks wrote:

> On Thu, Jan 17, 2002 at 08:37:52PM -0500, henken wrote:
> > I have tracked down an error message that shows up everytime bpsh hangs:
> > Jan 29 12:57:53 master bpmaster: write(ghost): missing process for message
> > type 14 req; to=3,27978 from=1,27978 result=0
> > Jan 29 16:20:06 master bpmaster: write(ghost): missing process for message
> > type 14 req; to=3,31128 from=1,31128 result=11
>
> Do these PIDs you're seeing match the ones that the master daemon is
> blowing up on?  I don't know how they'd be related but it's another
> data point.
>
> You might want to see if this fix to the master daemon's error
> handling helps you out.  (apply to daemons/master.c) It's for a
> problem I ran across last week and could result in the master failing
> to note a new process's location.  That could certainly lead to the
> above messages and maybe the other problem you're seeing.
>
> (This fix will be in 3.1.6)
>
> diff -u -r1.104 -r1.105
> --- master.c	2 Jan 2002 22:36:30 -0000	1.104
> +++ master.c	11 Jan 2002 03:05:45 -0000	1.105
> @@ -17,7 +17,7 @@
>   *  along with this program; if not, write to the Free Software
>   *  Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
>   *
> - * $Id: master.c,v 1.104 2002/01/02 22:36:30 hendriks Exp $
> + * $Id: master.c,v 1.105 2002/01/11 03:05:45 hendriks Exp $
>   *-----------------------------------------------------------------------*/
>  #include <sys/types.h>
>  #include <sys/stat.h>
> @@ -1446,9 +1446,14 @@
>  	    assoc->req = 0;	/* clear outstanding request */
>  	}
>      } else {
> -	if (req->req.fromtype == BPROC_ROUTE_REAL) {
> -	    /* Don't ever generate responses to these in case of slave errors */
> +	if (req->req.fromtype == BPROC_ROUTE_REAL &&
> +	    req->req.totype   != BPROC_ROUTE_GHOST) {
> +	    /* Don't make note of requests to ghosts...  We will never
> +	     * have to generate an error response due to ghost
> +	     * disappearance. */
>  	    switch(req->req.req) {
> +		/* Don't make note of these because we don't ever want
> +		 * to auto-generate responses to these messages */
>  	    case BPROC_GET_STATUS:
>  	    case BPROC_PARENT_EXIT:
>  		break;
>
>
> - Erik
>

-- 
Nicholas Henke
Undergraduate - Engineerring 2002
--
Senior Architect and Developer
Liniac Project - University of Pennsylvania
http://clubmask.sourceforge.net
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
There's nothing like good food, good beer, and a bad girl.