trillek / entropy

Entropy kernel for DCPU-16
8 stars 6 forks source link

Filesystem #13

Open ColErr opened 12 years ago

ColErr commented 12 years ago

I just wanted to see how big the FS code was compiled, and we are looking at 1Kw. I was just wanting to know if A) If this is about right for a filesystem B) If the first version of the code is just inefficient, or C) If the filesystem itself is contributing to the inefficiency.

With a quick look, about 200 words of that is converting 11char/11word -> 11char/5word and back again

Just wanted to make sure we are taking efficiency (size and speed) into consideration now, before we end up with windows Vista and not Linux

M52 commented 12 years ago

I like the Vista linux comparison. We should definitely be aiming for Linux efficiency. The horror of Vista on a low-end CPU like this is too much for me!!

All that aside, I think it could be optimized. It is just that we need to verify that it works 100%. If it does THEN we can start optimizing. Prelimary optimization is the root of evil, especially in error-prone languages like assembly.

Could you copy 'n paste the region that you think could be optimized? I personally think it is best if we look at this together and preferably with the author (Liraalr, right?) checking it too...

Liraal commented 12 years ago

Well, name conversion was programmed for speed not size, but Lucus has already suggested a better option, just no one had time to implement it yet. The rest of the code is in my opinion necessary. Bear in mind that much may yet change, because many both high- and low-level routines have been marked for a rewrite. M52 is right that I don't want to optimize it just yet. It's actually my fourth attempt to create a filesystem from the scratch. This code is designed to be modular, meaning it's split into layers of complexity and a layer can function without any 'higher' layers present.

Another thing is that this is not in fact the most recent version of it, because I have encountered some pretty weird errors and am working to fix them before I push it.

Also, the code itself is not inefficient, just big. In terms of speed, I think it's pretty quick.

dilbertfan commented 12 years ago

What exactly was Lucus' idea for 11/5?

Liraal commented 12 years ago

to make it 12/6 :)

dilbertfan commented 12 years ago

lol.png so a 9.3 FS?

Liraal commented 12 years ago

Explain.

dilbertfan commented 12 years ago

11=8+3 and 8.3 is a DOS type file naming system