Changed cache format to utilize better interframe encoding

and thus require sufficiently less hard disk space.
Instead of the old (plane_width/16)*(plane_height/16) blocksize,
now we are using 16x16 for the y plane and 8x8 blocks for the u,v planes.
Also this byte alignment allows comparing of blocks to happen by
first casting into a larger datatype(u_int64_t if available, u_int32_t else).
This way we do as little as 1/8 or 1/4 of comparisons. This essentially offsets
the higher CPU required to check the now increased number of blocks, and even
provides a slight performance boost from the previous way of caching.
Also, the load cache loop has been debloated, by moving much of the functionality
that previously resided in an if evaluation statement, into anew function(ReadFrame).
For further abstraction, ReadFrame utilizes ReadZF to do the actual reading.
ReadZF determines if a gzread or an fread has to be done.
Another improvement in the program is that, now, write operations on the disk, happen
in 4 kbytes pages, minimizing thus the overall cost of access.
Last change in this set is that the indexes in the CachedFrame and FrameHeader datatypes,
are now u_int32_t's, to accomodate the possibly large number of blocks(could have been 16 bit
types but that would give 4096*4096 max resolution, which might have caused problems in the future)


git-svn-id: https://recordmydesktop.svn.sourceforge.net/svnroot/recordmydesktop/trunk@266 f606c939-3180-4ac9-a4b8-4b8779d57d0a

1 files changed, 36 insertions, 11 deletions

diff --git a/recordmydesktop/include/rmdmacro.h b/recordmydesktop/include/rmdmacro.h
index 87ac39f..c14785f 100644
--- a/recordmydesktop/include/rmdmacro.h
+++ b/recordmydesktop/include/rmdmacro.h
@@ -69,8 +69,35 @@
 //avoid problems (amd64 has 8byte ulong)
 #define RMD_ULONG_SIZE_T (sizeof(unsigned long))
 
+//size of stride when comparing planes(depending on type)
+//this is just to avoid thousands of sizeof's
+#ifdef HAVE_U_INT64_T
+    #define COMPARE_STRIDE  8
+#else
+    #define COMPARE_STRIDE  4
+#endif
+
 //500 mb file size
 #define CACHE_FILE_SIZE_LIMIT (500*1<<20)
+//minimize hard disk access
+#define CACHE_OUT_BUFFER_SIZE 4096
+
+
+//The width, in bytes, of the blocks
+//on which the y,u and v planes are broken.
+//These blocks are square.
+#define Y_UNIT_WIDTH    0x0010
+#define UV_UNIT_WIDTH   0x0008
+
+//The number of bytes for every
+//sub-block of the y,u and v planes.
+//Since the blocks are square
+//these are obviously the squares
+//of the widths(specified above),
+//but the definitions bellow are only
+//for convenience anyway.
+#define Y_UNIT_BYTES    0x0100
+#define UV_UNIT_BYTES   0x0040
 
 
 #define CLIP_EVENT_AREA(e,brwin,wgeom){\
@@ -251,10 +278,10 @@
                        __copy_type,\
                        __bit_depth__){ \
     int k,i;\
-    register RMD_TYPE_##__bit_depth__ t_val;\
+    register u_int##__bit_depth__##_t t_val;\
     register unsigned char  *yuv_y=yuv->y+x_tm+y_tm*yuv->y_width,\
                             *_yr=Yr,*_yg=Yg,*_yb=Yb;\
-    register RMD_TYPE_##__bit_depth__ *datapi=(RMD_TYPE_##__bit_depth__ *)data\
+    register u_int##__bit_depth__##_t *datapi=(u_int##__bit_depth__##_t *)data\
                                               +((__copy_type==__X_SHARED)?\
                                               (x_tm+y_tm*yuv->y_width):0);\
     for(k=0;k<height_tm;k++){\
@@ -282,12 +309,12 @@
                          __sampling_type,\
                          __bit_depth__){  \
     int k,i;\
-    register RMD_TYPE_##__bit_depth__ t_val;\
+    register u_int##__bit_depth__##_t t_val;\
     register unsigned char  *yuv_u=yuv->u+x_tm/2+(y_tm*yuv->uv_width)/2,\
                             *yuv_v=yuv->v+x_tm/2+(y_tm*yuv->uv_width)/2,\
                             *_ur=Ur,*_ug=Ug,*_ub=Ub,\
                             *_vr=Vr,*_vg=Vg,*_vb=Vb;\
-    register RMD_TYPE_##__bit_depth__ *datapi=(RMD_TYPE_##__bit_depth__ *)data\
+    register u_int##__bit_depth__##_t *datapi=(u_int##__bit_depth__##_t *)data\
                                                +((__copy_type==__X_SHARED)?\
                                                 (x_tm+y_tm*yuv->y_width):0),\
                                       *datapi_next=NULL;\
@@ -445,20 +472,18 @@
     free(t_buf);\
 };\
 
-#define INIT_FRAME(frame_t,fheader_t,yuv_t){\
+#define INIT_FRAME(frame_t,fheader_t,yuv_t,\
+                   YBlocks_t,UBlocks_t,VBlocks_t){\
     (frame_t)->header=(fheader_t);\
-    (frame_t)->YBlocks=malloc(256);\
-    (frame_t)->UBlocks=malloc(64);\
-    (frame_t)->VBlocks=malloc(64);\
+    (frame_t)->YBlocks=YBlocks_t;\
+    (frame_t)->UBlocks=UBlocks_t;\
+    (frame_t)->VBlocks=VBlocks_t;\
     (frame_t)->YData=malloc((yuv_t)->y_width*(yuv_t)->y_height);\
     (frame_t)->UData=malloc((yuv_t)->uv_width*(yuv_t)->uv_height);\
     (frame_t)->VData=malloc((yuv_t)->uv_width*(yuv_t)->uv_height);\
 };
 
 #define CLEAR_FRAME(frame_t){\
-    free((frame_t)->YBlocks);\
-    free((frame_t)->UBlocks);\
-    free((frame_t)->VBlocks);\
     free((frame_t)->YData);\
     free((frame_t)->UData);\
     free((frame_t)->VData);\