[PATCH 0/2] CNFSv4: move to 4K blocksize
Russ Allbery
rra at stanford.edu
Sat Nov 29 21:10:30 UTC 2008
list-inn-workers at news.cistron.nl ("Miquel van Smoorenburg" ) writes:
> The values in the article header are stored in network byte order, but
> 'long' can be 32 or 64 bits (and so can time_t). I think we should make
> 'size' an uint32_t, 'arrived' an int32_t+uint32_t (upper and lower 32
> bits) and 'class' an uint32_t as well.
That sounds good to me.
I'm not sure if the additional work to avoid int64_t is worth it at this
point or if we should just add autoconf logic to find a 64-bit type in
case they don't have the C99 typedef but require a 64-bit type. I think
all modern OS and compiler combinations have one at this point. But it
would be the first place in INN that we assume that.
> Is it a good idea to remove the OLD_CNFS code ?
Yes, please.
--
Russ Allbery (rra at stanford.edu) <http://www.eyrie.org/~eagle/>
Please send questions to the list rather than mailing me directly.
<http://www.eyrie.org/~eagle/faqs/questions.html> explains why.
More information about the inn-workers
mailing list