X-Git-Url: https://git.ucc.asn.au/?a=blobdiff_plain;f=KernelLand%2FModules%2FStorage%2FLVM%2Fvolumes.c;h=176826768c946c94b689fa68ff2450a5fc8e5a22;hb=c8f776d4714fbbe3e96ad02bbc4331638deda0af;hp=af1986868ae101ec8caa2dd107ec876586cb176d;hpb=476abc0092cfb3ca93e8981f9554b5c7d2b53fe0;p=tpg%2Facess2.git

diff --git a/KernelLand/Modules/Storage/LVM/volumes.c b/KernelLand/Modules/Storage/LVM/volumes.c
index af198686..17682676 100644
--- a/KernelLand/Modules/Storage/LVM/volumes.c
+++ b/KernelLand/Modules/Storage/LVM/volumes.c
@@ -5,10 +5,12 @@
  * volumes.c
  * - Volume management
  */
-#define DEBUG	1
+#define DEBUG	0
 #include "lvm_int.h"
+#define USE_IOCACHE	1
 
 // === PROTOTYPES ===
+ int	LVM_int_CacheWriteback(void *ID, Uint64 Sector, const void *Buffer);
  int	LVM_int_VFSReadEmul(void *Arg, Uint64 BlockStart, size_t BlockCount, void *Dest);
  int	LVM_int_VFSWriteEmul(void *Arg, Uint64 BlockStart, size_t BlockCount, const void *Source);
 
@@ -38,6 +40,7 @@ void *LVM_AddVolume(const tLVM_VolType *Type, const char *Name, void *Ptr, size_
 	dummy_vol.Ptr = Ptr;
 	dummy_vol.BlockCount = BlockCount;
 	dummy_vol.BlockSize = BlockSize;
+	dummy_vol.CacheHandle = NULL;
 
 	// Read the first block of the volume	
 	void *first_block = malloc(BlockSize);
@@ -57,6 +60,8 @@ void *LVM_AddVolume(const tLVM_VolType *Type, const char *Name, void *Ptr, size_
 
 	// Type->CountSubvolumes
 	dummy_vol.nSubVolumes = fmt->CountSubvolumes(&dummy_vol, first_block);
+	Log_Debug("LVM", "Volume %s as format %s gives %i sub-volumes",
+		Name, fmt->Name, dummy_vol.nSubVolumes);
 	
 	// Create real volume descriptor
 	// TODO: If this needs to be rescanned later, having the subvolume list separate might be an idea
@@ -88,6 +93,14 @@ void *LVM_AddVolume(const tLVM_VolType *Type, const char *Name, void *Ptr, size_
 	real_vol->VolNode.ImplPtr = real_vol;
 	real_vol->VolNode.Size = BlockCount * BlockSize;
 
+	// TODO: Better selection of cache size
+	// TODO: Allow a volume type to disallow caching
+	#if USE_IOCACHE
+	real_vol->CacheHandle = IOCache_Create(LVM_int_CacheWriteback, real_vol, BlockSize, 1024);
+	#else
+	real_vol->CacheHandle = NULL;
+	#endif
+
 	// Type->PopulateSubvolumes
 	fmt->PopulateSubvolumes(real_vol, first_block);
 	free(first_block);
@@ -141,14 +154,65 @@ void LVM_int_SetSubvolume_Anon(tLVM_Vol *Volume, int Index, Uint64 FirstBlock, U
 // --------------------------------------------------------------------
 // IO
 // --------------------------------------------------------------------
+int LVM_int_CacheWriteback(void *ID, Uint64 Sector, const void *Buffer)
+{
+	tLVM_Vol *Volume = ID;
+	return Volume->Type->Write(Volume->Ptr, Sector, 1, Buffer);
+}
+
 size_t LVM_int_ReadVolume(tLVM_Vol *Volume, Uint64 BlockNum, size_t BlockCount, void *Dest)
 {
-	return Volume->Type->Read(Volume->Ptr, BlockNum, BlockCount, Dest);
+	#if USE_IOCACHE
+	if( Volume->CacheHandle )
+	{
+		 int	done = 0;
+		while( done < BlockCount )
+		{
+			while( done < BlockCount && IOCache_Read(Volume->CacheHandle, BlockNum+done, Dest) == 1 )
+				done ++, Dest = (char*)Dest + Volume->BlockSize;
+			size_t first_uncached = done;
+			void *uncache_buf = Dest;
+			LOG("%i/%i: cached", done, BlockCount);
+			while( done < BlockCount && IOCache_Read(Volume->CacheHandle, BlockNum+done, Dest) == 0 )
+				done ++, Dest = (char*)Dest + Volume->BlockSize;
+			LOG("%i/%i: uncached", done, BlockCount);
+			size_t	count = done-first_uncached;
+			if( count ) {
+				Volume->Type->Read(Volume->Ptr, BlockNum+first_uncached, count, uncache_buf);
+				while(count--)
+				{
+					IOCache_Add(Volume->CacheHandle, BlockNum+first_uncached, uncache_buf);
+					first_uncached ++;
+					uncache_buf = (char*)uncache_buf + Volume->BlockSize;
+				}
+			}
+		}
+		return done;
+	}
+	else
+	#endif
+		return Volume->Type->Read(Volume->Ptr, BlockNum, BlockCount, Dest);
 }
 
 size_t LVM_int_WriteVolume(tLVM_Vol *Volume, Uint64 BlockNum, size_t BlockCount, const void *Src)
 {
-	return Volume->Type->Write(Volume->Ptr, BlockNum, BlockCount, Src);	
+	#if USE_IOCACHE
+	if( Volume->CacheHandle )
+	{
+		int done = 0;
+		while( BlockCount )
+		{
+			IOCache_Write(Volume->CacheHandle, BlockNum, Src);
+			Src = (const char*)Src + Volume->BlockSize;
+			BlockNum ++;
+			BlockCount --;
+			done ++;
+		}
+		return done;
+	}
+	else
+	#endif
+		return Volume->Type->Write(Volume->Ptr, BlockNum, BlockCount, Src);
 }
 
 int LVM_int_VFSReadEmul(void *Arg, Uint64 BlockStart, size_t BlockCount, void *Dest)