a mutex added to TempMods which hopefully fixes rare segfaults on client

diff --git a/Makefile b/Makefile
index 6da2e64d1dfef545868b7da5d3fc08a6a7a42214..5dcee0d9a7beac69af7751f80c39fc5e8b191c3b 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -26,18 +26,12 @@ CXXFLAGS = -O2 -ffast-math -Wall -g -pipe
 #CXXFLAGS = -O1 -ffast-math -Wall -g\r
 #CXXFLAGS = -Wall -g -O0\r
 \r
-#FAST_CXXFLAGS = -O3 -ffast-math -Wall -fomit-frame-pointer -pipe -funroll-loops -mtune=i686\r
-\r
-#Default target\r
-\r
-all: all_linux\r
+all: fast_linux\r
 \r
 ifeq ($(HOSTTYPE), x86_64)\r
 LIBSELECT=64\r
 endif\r
 \r
-# Target specific settings\r
-\r
 all_linux fast_linux: LDFLAGS = -L/usr/X11R6/lib$(LIBSELECT) -L$(IRRLICHTPATH)/lib/Linux -L$(JTHREADPATH)/src/.libs -lIrrlicht -lGL -lXxf86vm -lXext -lX11 -ljthread -lz\r
 all_linux fast_linux: CPPFLAGS = -I$(IRRLICHTPATH)/include -I/usr/X11R6/include -I$(JTHREADPATH)/src\r
 fast_linux server_linux: CXXFLAGS = -O3 -ffast-math -Wall -fomit-frame-pointer -pipe -funroll-loops -mtune=i686\r
@@ -45,20 +39,13 @@ server_linux: LDFLAGS = -L$(JTHREADPATH)/src/.libs -ljthread -lz -lpthread
 server_linux: CPPFLAGS = -I$(IRRLICHTPATH)/include -I/usr/X11R6/include -I$(JTHREADPATH)/src -DSERVER\r
 all_linux fast_linux clean_linux: SYSTEM=Linux\r
 \r
-# These are out of date\r
-all_win32: LDFLAGS = -L$(IRRLICHTPATH)/lib/Win32-gcc -L$(JTHREADPATH)/Debug -lIrrlicht -lopengl32 -lm -ljthread\r
-all_win32 clean_win32: SYSTEM=Win32-gcc\r
-all_win32 clean_win32: SUF=.exe\r
-\r
-# Name of the binary - only valid for targets which set SYSTEM\r
-\r
-DESTPATH = bin/$(TARGET)$(SUF)\r
-FAST_DESTPATH = bin/$(FAST_TARGET)$(SUF)\r
-SERVER_DESTPATH = bin/$(SERVER_TARGET)$(SUF)\r
+DESTPATH = bin/$(TARGET)\r
+FAST_DESTPATH = bin/$(FAST_TARGET)\r
+SERVER_DESTPATH = bin/$(SERVER_TARGET)\r
 \r
 # Build commands\r
 \r
-all_linux all_win32: $(BUILD_DIR) $(DESTPATH)\r
+all_linux: $(BUILD_DIR) $(DESTPATH)\r
 fast_linux: $(FAST_BUILD_DIR) $(FAST_DESTPATH)\r
 server_linux: $(SERVER_BUILD_DIR) $(SERVER_DESTPATH)\r
 \r
@@ -87,13 +74,13 @@ $(FAST_BUILD_DIR)/%.o: src/%.cpp
 $(SERVER_BUILD_DIR)/%.o: src/%.cpp\r
        $(CXX) -c -o $@ $< $(CPPFLAGS) $(CXXFLAGS)\r
 \r
-clean: clean_linux clean_win32 clean_fast_linux clean_server_linux\r
+clean: clean_linux clean_fast_linux clean_server_linux\r
 \r
-clean_linux clean_win32:\r
+clean_linux:\r
        @$(RM) $(OBJECTS) $(DESTPATH)\r
 \r
 clean_fast_linux:\r
-       @$(RM) $(OBJECTS) $(FAST_DESTPATH)\r
+       @$(RM) $(FAST_OBJECTS) $(FAST_DESTPATH)\r
 \r
 clean_server_linux:\r
        @$(RM) $(SERVER_OBJECTS) $(SERVER_DESTPATH)\r

diff --git a/src/main.cpp b/src/main.cpp
index 5ffbafb62709852d1c1a23498c0ba6f55df0e281..5d6712df6e1fbc6e819d39a61da8d214631e77ee 100644 (file)
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -91,6 +91,7 @@ SUGG: Implement a "Fast check queue" (a queue with a map for checking
 \r
 SUGG: Signs could be done in the same way as torches. For this, blocks\r
       need an additional metadata field for the texts\r
+         - This is also needed for item container chests\r
 \r
 SUGG: Precalculate lighting translation table at runtime (at startup)\r
 \r
@@ -99,6 +100,15 @@ SUGG: A version number to blocks, which increments when the block is
          - This can then be used to make sure the most recent version of\r
            a block has been sent to client\r
 \r
+SUGG: Make the amount of blocks sending to client and the total\r
+         amount of blocks dynamically limited. Transferring blocks is the\r
+         main network eater of this system, so it is the one that has\r
+         to be throttled so that RTTs stay low.\r
+\r
+TODO: Untie client network operations from framerate\r
+      - Needs some input queues or something\r
+         - Not really necessary?\r
+\r
 TODO: Combine MapBlock's face caches to so big pieces that VBO\r
       gets used\r
       - That is >500 vertices\r
@@ -106,11 +116,7 @@ TODO: Combine MapBlock's face caches to so big pieces that VBO
 TODO: Better dungeons\r
 TODO: Cliffs, arcs\r
 \r
-TODO: Menus\r
-\r
-TODO: Moving players more smoothly. Calculate moving animation\r
-      in a way that doesn't make the player jump to the right place\r
-         immediately when the server sends a new position\r
+TODO: Startup and configuration menu\r
 \r
 TODO: There are some lighting-related todos and fixmes in\r
       ServerMap::emergeBlock\r
@@ -119,16 +125,10 @@ TODO: Proper handling of spawning place (try to find something that
       is not in the middle of an ocean (some land to stand on at\r
          least) and save it in map config.\r
 \r
-TODO: Make the amount of blocks sending to client and the total\r
-         amount of blocks dynamically limited. Transferring blocks is the\r
-         main network eater of this system, so it is the one that has\r
-         to be throttled so that RTTs stay low.\r
-\r
-TODO: Server to load starting inventory from disk\r
-\r
 TODO: Players to only be hidden when the client quits.\r
 TODO: - Players to be saved on disk, with inventory\r
 TODO: Players to be saved as text in map/players/<name>\r
+TODO: Player inventory to be saved on disk\r
 \r
 TODO: Make fetching sector's blocks more efficient when rendering\r
       sectors that have very large amounts of blocks (on client)\r
@@ -149,9 +149,6 @@ Block object server side:
 TODO: Copy the text of the last picked sign to inventory in creative\r
       mode\r
 \r
-TODO: Untie client network operations from framerate\r
-      - Needs some input queues or something\r
-\r
 TODO: Get rid of GotSplitPacketException\r
 \r
 TODO: Check what goes wrong with caching map to disk (Kray)\r
@@ -168,7 +165,7 @@ TODO: Better handling of objects and mobs
            - Make other players utilize the same framework\r
 \r
 TODO: Draw big amounts of torches better (that is, throw them in the\r
-      same meshbuffer (can the meshcombiner class be used?))\r
+      same meshbuffer (can the meshcollector class be used?))\r
 \r
 Doing now:\r
 ======================================================================\r

diff --git a/src/mapblock.cpp b/src/mapblock.cpp
index b2b5bc4f44f9fb99f67e98419ec0e6c06c20214c..186256589fded7eaac1514379d2701b151adf5b7 100644 (file)
--- a/src/mapblock.cpp
+++ b/src/mapblock.cpp
@@ -47,6 +47,7 @@ MapBlock::MapBlock(NodeContainer *parent, v3s16 pos, bool dummy):
        m_mesh_expired = false;
        mesh_mutex.Init();
        mesh = NULL;
+       m_temp_mods_mutex.Init();
 #endif
 }
 
@@ -584,47 +585,52 @@ void MapBlock::updateMesh(u32 daynight_ratio)
 
                NOTE: This is the slowest part of this method.
        */
+       
+       {
+               // Lock this, as m_temp_mods will be used directly
+               JMutexAutoLock lock(m_temp_mods_mutex);
 
-       /*
-               Go through every y,z and get top faces in rows of x+
-       */
-       for(s16 y=0; y<MAP_BLOCKSIZE; y++){
-               for(s16 z=0; z<MAP_BLOCKSIZE; z++){
-                       updateFastFaceRow(daynight_ratio, posRelative_f,
-                                       v3s16(0,y,z), MAP_BLOCKSIZE,
-                                       v3s16(1,0,0), //dir
-                                       v3f  (1,0,0),
-                                       v3s16(0,1,0), //face dir
-                                       v3f  (0,1,0),
-                                       fastfaces_new);
-               }
-       }
-       /*
-               Go through every x,y and get right faces in rows of z+
-       */
-       for(s16 x=0; x<MAP_BLOCKSIZE; x++){
+               /*
+                       Go through every y,z and get top faces in rows of x+
+               */
                for(s16 y=0; y<MAP_BLOCKSIZE; y++){
-                       updateFastFaceRow(daynight_ratio, posRelative_f,
-                                       v3s16(x,y,0), MAP_BLOCKSIZE,
-                                       v3s16(0,0,1),
-                                       v3f  (0,0,1),
-                                       v3s16(1,0,0),
-                                       v3f  (1,0,0),
-                                       fastfaces_new);
+                       for(s16 z=0; z<MAP_BLOCKSIZE; z++){
+                               updateFastFaceRow(daynight_ratio, posRelative_f,
+                                               v3s16(0,y,z), MAP_BLOCKSIZE,
+                                               v3s16(1,0,0), //dir
+                                               v3f  (1,0,0),
+                                               v3s16(0,1,0), //face dir
+                                               v3f  (0,1,0),
+                                               fastfaces_new);
+                       }
                }
-       }
-       /*
-               Go through every y,z and get back faces in rows of x+
-       */
-       for(s16 z=0; z<MAP_BLOCKSIZE; z++){
-               for(s16 y=0; y<MAP_BLOCKSIZE; y++){
-                       updateFastFaceRow(daynight_ratio, posRelative_f,
-                                       v3s16(0,y,z), MAP_BLOCKSIZE,
-                                       v3s16(1,0,0),
-                                       v3f  (1,0,0),
-                                       v3s16(0,0,1),
-                                       v3f  (0,0,1),
-                                       fastfaces_new);
+               /*
+                       Go through every x,y and get right faces in rows of z+
+               */
+               for(s16 x=0; x<MAP_BLOCKSIZE; x++){
+                       for(s16 y=0; y<MAP_BLOCKSIZE; y++){
+                               updateFastFaceRow(daynight_ratio, posRelative_f,
+                                               v3s16(x,y,0), MAP_BLOCKSIZE,
+                                               v3s16(0,0,1),
+                                               v3f  (0,0,1),
+                                               v3s16(1,0,0),
+                                               v3f  (1,0,0),
+                                               fastfaces_new);
+                       }
+               }
+               /*
+                       Go through every y,z and get back faces in rows of x+
+               */
+               for(s16 z=0; z<MAP_BLOCKSIZE; z++){
+                       for(s16 y=0; y<MAP_BLOCKSIZE; y++){
+                               updateFastFaceRow(daynight_ratio, posRelative_f,
+                                               v3s16(0,y,z), MAP_BLOCKSIZE,
+                                               v3s16(1,0,0),
+                                               v3f  (1,0,0),
+                                               v3s16(0,0,1),
+                                               v3f  (0,0,1),
+                                               fastfaces_new);
+                       }
                }
        }
 

diff --git a/src/mapblock.h b/src/mapblock.h
index dc077c23f4acc358042adfec9a306ce24d3c7ef0..586c10228f1f36e0880bc49c297b776b899be6d9 100644 (file)
--- a/src/mapblock.h
+++ b/src/mapblock.h
@@ -399,15 +399,30 @@ public:
                                <<", mod.type="<<mod.type
                                <<", mod.param="<<mod.param
                                <<std::endl;*/
+               JMutexAutoLock lock(m_temp_mods_mutex);
                m_temp_mods[p] = mod;
        }
+       // Returns true if there was one
+       bool getTempMod(v3s16 p, struct NodeMod *mod)
+       {
+               JMutexAutoLock lock(m_temp_mods_mutex);
+               core::map<v3s16, NodeMod>::Node *n;
+               n = m_temp_mods.find(p);
+               if(n == NULL)
+                       return false;
+               if(mod)
+                       *mod = n->getValue();
+               return true;
+       }
        void clearTempMod(v3s16 p)
        {
+               JMutexAutoLock lock(m_temp_mods_mutex);
                if(m_temp_mods.find(p))
                        m_temp_mods.remove(p);
        }
        void clearTempMods()
        {
+               JMutexAutoLock lock(m_temp_mods_mutex);
                m_temp_mods.clear();
        }
 #endif
@@ -517,6 +532,7 @@ private:
        // Temporary modifications to nodes
        // These are only used when drawing
        core::map<v3s16, NodeMod> m_temp_mods;
+       JMutex m_temp_mods_mutex;
 #endif
 };