--- deliantra/server/socket/loop.C 2008/04/22 07:01:47 1.73
+++ deliantra/server/socket/loop.C 2012/01/03 11:25:37 1.92
@@ -1,22 +1,21 @@
/*
* This file is part of Deliantra, the Roguelike Realtime MMORPG.
*
- * Copyright (©) 2005,2006,2007 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
- * Copyright (©) 2002-2003,2007 Mark Wedel & The Crossfire Development Team
- * Copyright (©) 1992,2007 Frank Tore Johansen
+ * Copyright (©) 2005,2006,2007,2008,2009,2010,2011,2012 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
*
- * Deliantra is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * Deliantra is free software: you can redistribute it and/or modify it under
+ * the terms of the Affero GNU General Public License as published by the
+ * Free Software Foundation, either version 3 of the License, or (at your
+ * option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
- * You should have received a copy of the GNU General Public License
- * along with this program. If not, see .
+ * You should have received a copy of the Affero GNU General Public License
+ * and the GNU General Public License along with this program. If not, see
+ * .
*
* The authors can be reached via e-mail to
*/
@@ -44,16 +43,11 @@
#include
#include
-#include
-
#define BG_SCRUB_RATE 4 // how often to send a face in the background
#define MAX_QUEUE_DEPTH 50
#define MAX_QUEUE_BACKLOG 3.
-// disconnect a socket after this many seconds without an ack
-#define SOCKET_TIMEOUT 16.
-
void
client::reset_state ()
{
@@ -71,7 +65,6 @@
if (cmd_queue.size () >= MAX_QUEUE_DEPTH)
{
- //TODO: just disconnect here?
reset_state ();
send_packet_printf ("drawinfo %d command queue overflow, ignoring.", NDI_RED);
}
@@ -97,8 +90,16 @@
if (cmd.stamp + MAX_QUEUE_BACKLOG < NOW)
{
- reset_state ();
- send_packet_printf ("drawinfo %d ignoring delayed commands.", NDI_RED);
+ reset_state (); // the command might actually reset some movement state etc.
+
+ if (pl)
+ pl->failmsg (
+ "Cannot keep up with your commands, ignoring them! "
+ "H. "
+ "Try issuing commands slower, or, if G is incapacitated (paralyzed and so on), "
+ "wait till your character can act again.\n\nIf G is permanently stuck, then "
+ "try the B command (or use B to ask somebody to B you out).>"
+ );
}
else
execute (cmd.handler, cmd.data, cmd.datalen);
@@ -117,20 +118,24 @@
if (!pl || destroyed ())
return;
+ pl->dirty = true;
+
/* Update the players stats once per tick. More efficient than
* sending them whenever they change, and probably just as useful
*/
+ pl->need_updated_stats ();
esrv_update_stats (pl);
-#if 0
- if (last_weight != -1 && last_weight != pl->ob->client_weight ())
+ if (pl->ns->update_spells)
+ esrv_update_spells (pl);
+
+ sint32 weight = pl->ob->client_weight ();
+
+ if (last_weight != weight)
{
+ pl->ob->update_stats ();
esrv_update_item (UPD_WEIGHT, pl->ob, pl->ob);
- if (last_weight != pl->ob->client_weight ())
- LOG (llevError, "esrv_update_item(UPD_WEIGHT) did not set player weight: is %lu, should be %lu\n",
- (unsigned long) last_weight, pl->ob->client_weight ());
}
-#endif
draw_client_map (pl);
@@ -160,10 +165,10 @@
tcpi.tcpi_snd_cwnd - (tcpi.tcpi_unacked - tcpi.tcpi_sacked));
#endif
- // fast-time-out a player by checking for missign acks
+ // fast-time-out a player by checking for missing acks
// do this only when player is active
if (pl && pl->active
- && tcpi.tcpi_last_ack_recv > int (SOCKET_TIMEOUT * 1000))
+ && tcpi.tcpi_last_ack_recv > int (socket_timeout * 1000))
{
send_msg (NDI_RED | NDI_REPLY, "connection-timeout", "safety disconnect due to tcp/ip timeout (no packets received)");
write_outputbuffer ();
@@ -175,22 +180,23 @@
}
#endif
- rate_avail = min (max_rate + mss, rate_avail + max_rate);
+ // limit budget surplus/deficit by one mss, add per-tick budget
+ rate_avail = min (rate_avail, mss) + max_rate;
int max_send = rate_avail;
#if HAVE_TCP_INFO
// further restrict the available bandwidth by the excess bandwidth available
- max_send = min (max_send, (tcpi.tcpi_snd_cwnd - tcpi.tcpi_unacked + tcpi.tcpi_sacked) * mss);
+ min_it (max_send, (tcpi.tcpi_snd_cwnd - tcpi.tcpi_unacked + tcpi.tcpi_sacked) * mss);
#endif
- // if we can split images, round to next-lowest mss
- if (fxix) max_send -= max_send % mss;
+ // round to next-lowest mss
+ max_send -= max_send % mss;
if (ixface.empty ())
{
// regularly send a new face when queue is empty
- if (bg_scrub && !--bg_scrub && enable_bg_scrub)
+ if (bg_scrub && !--bg_scrub)
while (scrub_idx < faces.size () - 1)
{
++scrub_idx;
@@ -222,36 +228,28 @@
if (facedata *d = face_data (ix.idx, faceset))
{
- if (fxix)
- {
- // estimate the packet header overhead "ix " + idx + (new)ofs
- int pktlen = 3 + ber32::encoded_size (ix.idx) + ber32::encoded_size (ix.ofs);
- int chunk = min (avail - packet::hdrlen, MAXSOCKBUF) - pktlen;
-
- // only transfer something if the amount of data transferred
- // has a healthy relation to the header overhead
- if (chunk < 64)
- break;
+ // estimate the packet header overhead "ix " + idx + (new)ofs
+ int pktlen = 3 + ber32::encoded_size (ix.idx) + ber32::encoded_size (ix.ofs);
+ int chunk = min (avail - packet::hdrlen, MAXSOCKBUF) - pktlen;
+
+ // only transfer something if the amount of data transferred
+ // has a healthy relation to the header overhead
+ if (chunk < 64)
+ break;
- chunk = min (chunk, (int)ix.ofs);
+ chunk = min (chunk, (int)ix.ofs);
- ix.ofs -= chunk;
+ ix.ofs -= chunk;
- //fprintf (stderr, "i%dx %6d: %5d+%4d (%4d)\n", fxix, ix.idx,ix.ofs,chunk, ixface.size());//D
+ //fprintf (stderr, "i%dx %6d: %5d+%4d (%4d)\n", fxix, ix.idx,ix.ofs,chunk, ixface.size());//D
- packet sl ("ix");
+ packet sl ("ix");
- sl << ber32 (ix.idx)
- << ber32 (ix.ofs)
- << data (d->data.data () + ix.ofs, chunk);
+ sl << ber32 (ix.idx)
+ << ber32 (ix.ofs)
+ << data (d->data + ix.ofs, chunk);
- send_packet (sl);
- }
- else
- {
- send_image (ix.idx);
- ix.ofs = 0;
- }
+ send_packet (sl);
}
else
ix.ofs = 0;
@@ -270,14 +268,14 @@
}
void
-client::flush_sockets (void)
+client::flush_sockets ()
{
for (sockvec::iterator i = clients.begin (); i != clients.end (); ++i)
(*i)->flush ();
}
void
-client::clock (void)
+client::clock ()
{
for (sockvec::iterator i = clients.begin (); i != clients.end (); ++i)
(*i)->tick ();