[ViewVC] Diff of: cvs/deliantra/server/socket/loop.C

Comparing deliantra/server/socket/loop.C (file contents):
Revision 1.54 by root, Sun Jul 1 04:08:14 2007 UTC vs.
Revision 1.71 by root, Sun Jan 13 12:39:29 2008 UTC

 /*
- * This file is part of Crossfire TRT, the Multiplayer Online Role Playing Game.
+ * This file is part of Deliantra, the Roguelike Realtime MMORPG.
  *
- * Copyright (©) 2005,2006,2007 Marc Alexander Lehmann / Robin Redeker / the Crossfire TRT team
+ * 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
  *
- * Crossfire TRT is free software; you can redistribute it and/or modify it
+ * Deliantra is free software: you can redistribute it and/or modify
- * under the terms of the GNU General Public License as published by the Free
+ * it under the terms of the GNU General Public License as published by
- * Software Foundation; either version 2 of the License, or (at your option)
+ * the Free Software Foundation, either version 3 of the License, or
- * any later version.
+ * (at your option) any later version.
  *
- * This program is distributed in the hope that it will be useful, but
+ * This program is distributed in the hope that it will be useful,
- * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * for more details.
+ * GNU General Public License for more details.
  *
- * You should have received a copy of the GNU General Public License along
+ * You should have received a copy of the GNU General Public License
- * with Crossfire TRT; if not, write to the Free Software Foundation, Inc. 51
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
- * Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  *
- * The authors can be reached via e-mail to <crossfire@schmorp.de>
+ * The authors can be reached via e-mail to <support@deliantra.net>
  */
 /**
  * \file
  * Main client/server loops.
 #include <loader.h>
 #define BG_SCRUB_RATE 4 // how often to send a face in the background
-#define MAX_QUEUE_DEPTH   500
+#define MAX_QUEUE_DEPTH   50
 #define MAX_QUEUE_BACKLOG 3.
+// disconnect a socket after this many seconds without an ack
+#define SOCKET_TIMEOUT 8.
 void
 client::reset_state ()
 {
   if (!pl)
       reset_state ();
       send_packet_printf ("drawinfo %d command queue overflow, ignoring.", NDI_RED);
     }
   else
     {
-      cmd_queue.push_back (command ());
+      cmd_queue.resize (cmd_queue.size () + 1);
       command &cmd = cmd_queue.back ();
       cmd.stamp   = stamp;
       cmd.handler = handler;
       cmd.data    = salloc<char> (datalen + 1, data);
       cmd.datalen = datalen;
           send_packet_printf ("drawinfo %d ignoring delayed commands.", NDI_RED);
         }
       else
         execute (cmd.handler, cmd.data, cmd.datalen);
+      sfree<char> (cmd.data, cmd.datalen + 1);
       cmd_queue.pop_front ();
       return true;
     }
   else
     return false;
-}
-void
-flush_sockets (void)
-{
-  for (sockvec::iterator i = clients.begin (); i != clients.end (); ++i)
-    (*i)->flush ();
-}
-/**
- * This checks the sockets for input, does the right thing.
- *
- * A bit of this code is grabbed out of socket.c
- * There are 2 lists we need to look through - init_sockets is a list
- *
- */
-void
-doeric_server (void)
-{
-  //TODO: should not be done here, either
-  for (unsigned i = 0; i < clients.size (); ++i)
-    {
-      client *ns = clients [i];
-      ns->tick ();
-      ns->refcnt_chk ();
-    }
 }
 void
 client::tick ()
 {
   draw_client_map (pl);
   if (update_look)
     esrv_draw_look (pl);
+  mapinfo_queue_run ();
+#if HAVE_TCP_INFO
+  // check time of last ack, and, if too old, kill connection
+  socklen_t len = sizeof (tcpi);
+  if (!getsockopt (fd, IPPROTO_TCP, TCP_INFO, &tcpi, &len) && len == sizeof (tcpi))
+    {
+      if (tcpi.tcpi_snd_mss)
+        mss = tcpi.tcpi_snd_mss;
+#if 0
+      fprintf (stderr, "uack %d ack %d lost %d ret %d fack %d sst %d cwnd %d mss %d pmtu %d advmss %d EXC %d\n",
+               tcpi.tcpi_unacked,
+               tcpi.tcpi_sacked,
+               tcpi.tcpi_lost,
+               tcpi.tcpi_retrans,
+               tcpi.tcpi_fackets,
+               tcpi.tcpi_snd_ssthresh, tcpi.tcpi_snd_cwnd, tcpi.tcpi_advmss, tcpi.tcpi_pmtu, tcpi.tcpi_advmss,
+               tcpi.tcpi_snd_cwnd - (tcpi.tcpi_unacked - tcpi.tcpi_sacked));
+#endif
+      // fast-time-out a player by checking for missign acks
+      // do this only when player is active
+      if (pl && pl->active
+          && 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 ();
+          LOG (llevDebug, "connection on fd %d closed due to ack timeout (%u/%u/%u)\n", fd,
+               (unsigned)tcpi.tcpi_last_ack_recv, (unsigned)tcpi.tcpi_last_data_sent, (unsigned)tcpi.tcpi_unacked);
+          destroy ();
+        }
+    }
+#endif
+  rate_avail = min (max_rate + mss, rate_avail + 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);
+#endif
+  // if we can split images, round to next-lowest mss
+  if (fxix) 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)
         while (scrub_idx < faces.size () - 1)
           {
             ++scrub_idx;
             if (!faces_sent [scrub_idx])
+              if (faceinfo *f = face_info (scrub_idx))
+                if (f->type == FT_FACE || f->type == FT_SOUND) // only scrub faces and sounds for now
-              {
+                  {
-                send_face (scrub_idx);
+                    send_face (scrub_idx, -120);
-                flush_fx ();
+                    flush_fx ();
-                bg_scrub = 1; // send up to one face per tick, unless an image was requested
+                    bg_scrub = 1; // send up to one fx per tick, unless an image was requested
-                break;
+                    break;
-              }
+                  }
           }
-      rate_avail = max_rate - outputbuffer_len ();
     }
   else
     {
-      int ol = outputbuffer_len ();
-      rate_avail = min (max_rate, rate_avail + max_rate);
-      rate_avail -= ol;
-      int avail = rate_avail;
-      // if we can split images, transfer up to mss-sized packets if possible
-      // but never 768 bytes more.
-      if (fxix) avail += min (768, mss - (ol % mss));
       bg_scrub = BG_SCRUB_RATE;
-      while (avail > 0)
+      for (;;)
         {
+          int avail = max_send - outputbuffer_len ();
+          if (avail <= 0)
+            break;
           ixsend &ix = ixface.back ();
           if (facedata *d = face_data (ix.idx, faceset))
             {
               if (fxix)
                 {
-                  // 9 bytes is enough for fx_FFFOOO, 40 leaves some room for image data
+                  // 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 (min (avail - 40, MAXSOCKBUF - 9), 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 <= 0)
+                  if (chunk < 64)
                     break;
+                  chunk = min (chunk, (int)ix.ofs);
                   ix.ofs -= chunk;
+                  //fprintf (stderr, "i%dx %6d: %5d+%4d   (%4d)\n", fxix, ix.idx,ix.ofs,chunk, ixface.size());//D
                   packet sl ("ix");
                   sl << ber32 (ix.idx)
                      << ber32 (ix.ofs)
                 }
             }
           else
             ix.ofs = 0;
-          int consumed = ol - outputbuffer_len ();
-          avail      -= consumed;
-          rate_avail -= consumed;
-          ol = outputbuffer_len ();
           if (!ix.ofs)
             {
               ixface.pop_back ();
               if (ixface.empty ())
                 break;
             }
         }
     }
-}
+  rate_avail -= outputbuffer_len ();
+}
+void
+client::flush_sockets (void)
+{
+  for (sockvec::iterator i = clients.begin (); i != clients.end (); ++i)
+    (*i)->flush ();
+}
+void
+client::clock (void)
+{
+  for (sockvec::iterator i = clients.begin (); i != clients.end (); ++i)
+    (*i)->tick ();
+  // give them all the same chances
+  flush_sockets ();
+  //TODO: should not be done here, either
+  for (unsigned i = 0; i < clients.size (); ++i)
+    clients[i]->refcnt_chk ();
+}

Diff Legend

-–
+Removed lines
-+
+Added lines
-<
+Changed lines
->
+Changed lines

Comparing deliantra/server/socket/loop.C (file contents): Revision 1.54 by root, Sun Jul 1 04:08:14 2007 UTC vs. Revision 1.71 by root, Sun Jan 13 12:39:29 2008 UTC

Diff Legend

Comparing deliantra/server/socket/loop.C (file contents):
Revision 1.54 by root, Sun Jul 1 04:08:14 2007 UTC vs.
Revision 1.71 by root, Sun Jan 13 12:39:29 2008 UTC