X-Git-Url: http://dxcluster.org/gitweb/gitweb.cgi?a=blobdiff_plain;f=perl%2FDXCluster.pm;h=aee8fe4d94a0c6a0e1c192fb1d94d836b705edd1;hb=c8040f10eceea03637274064458ffe976a98e285;hp=91c43a7880a8944d7fdbb0754bc7d8c1e58d0b35;hpb=1994ae9ca216a19bf373948c5e8d79a9d58369d6;p=spider.git

diff --git a/perl/DXCluster.pm b/perl/DXCluster.pm
index 91c43a78..aee8fe4d 100644
--- a/perl/DXCluster.pm
+++ b/perl/DXCluster.pm
@@ -131,17 +131,11 @@ sub cluster
 {
 	my $users = DXCommandmode::get_all();
 	my $uptime = main::uptime();
-	my $tot = $DXNode::users + 1;
+	my $tot = $DXNode::users;
 		
 	return " $DXNode::nodes nodes, $users local / $tot total users  Max users $DXNode::maxusers  Uptime $uptime";
 }
 
-sub DESTROY
-{
-	my $self = shift;
-	dbg('cluster', "destroying $self->{call}\n");
-}
-
 no strict;
 sub AUTOLOAD
 {
@@ -175,9 +169,8 @@ sub new
   
 	my $self = $pkg->alloc($dxchan, $call, $confmode, $here);
 	$self->{mynode} = $node;
-	$node->{list}->{$call} = $self;	# add this user to the list on this node
+	$node->add_user($call, $self);
 	dbg('cluster', "allocating user $call to $node->{call} in cluster\n");
-	$node->update_users();
 	return $self;
 }
 
@@ -187,10 +180,8 @@ sub del
 	my $call = $self->{call};
 	my $node = $self->{mynode};
 
-	delete $node->{list}->{$call};
-	delete $DXCluster::cluster{$call}; # remove me from the cluster table
+	$node->del_user($call);
 	dbg('cluster', "deleting user $call from $node->{call} in cluster\n");
-	$node->update_users();
 }
 
 sub count
@@ -254,7 +245,35 @@ sub del
 	}
 	delete $DXCluster::cluster{$call}; # remove me from the cluster table
 	dbg('cluster', "deleting node $call from cluster\n"); 
-	$nodes-- if $nodes > 0;
+	$users -= $self->{users};    # it may be PC50 updated only therefore > 0
+	$users = 0 if $users < 0;
+	$nodes--;
+	$nodes = 0 if $nodes < 0;
+}
+
+sub add_user
+{
+	my $self = shift;
+	my $call = shift;
+	my $ref = shift;
+	
+	$self->{list}->{$call} = $ref; # add this user to the list on this node
+	$self->{users} = keys %{$self->{list}};
+	$users++;
+	$maxusers = $users+$nodes if $users+$nodes > $maxusers;
+}
+
+sub del_user
+{
+	my $self = shift;
+	my $call = shift;
+
+	delete $self->{list}->{$call};
+	delete $DXCluster::cluster{$call}; # remove me from the cluster table
+	$self->{users} = keys %{$self->{list}};
+	$users--;
+	$users = 0, warn "\$users gone neg, reset" if $users < 0;
+	$maxusers = $users+$nodes if $users+$nodes > $maxusers;
 }
 
 sub update_users
@@ -262,14 +281,10 @@ sub update_users
 	my $self = shift;
 	my $count = shift;
 	$count = 0 unless $count;
-  
-	$users -= $self->{users} if $self->{users};
-	if ((keys %{$self->{list}})) {
-		$self->{users} = (keys %{$self->{list}});
-	} else {
-		$self->{users} = $count;
-	}
-	$users += $self->{users} if $self->{users};
+	
+	$users -= $self->{users};
+	$self->{users} = $count unless keys %{$self->{list}};
+	$users += $self->{users};
 	$maxusers = $users+$nodes if $users+$nodes > $maxusers;
 }
 
@@ -282,5 +297,13 @@ sub dolist
 {
 
 }
+
+sub DESTROY
+{
+	my $self = shift;
+	undef $self->{list} if $self->{list};
+}
+
+
 1;
 __END__