svn commit: r235601 - head/include/protocols

Garance A Drosehn gad at FreeBSD.org
Mon May 21 18:28:24 UTC 2012


On 5/21/12 11:27 AM, John Baldwin wrote:
> On Friday, May 18, 2012 11:24:36 am Gleb Kurtsou wrote:
>    
>> On (18/05/2012 09:41), John Baldwin wrote:
>>      
>>> A question about your stat changes: did you expand dev_t to 32 bits
>>> for the AFS folks, or did you leave it as 16 bits?
>> dev_t is already 32-bit. Changing it to 64-bit was discussed at some
>> point and from what I recall no decision was made:
>>
>>    http://marc.info/?t=129119478700005&r=1&w=2
>>
>> I'm going to commit preparatory changes only for now. Then publish
>> diff for testing. We can still change dev_t to 64-bit if needed.
>> Although I didn't work on it.
>>      
> Ah, it was 64-bit they asked for.  If it is easy to do so, I'd favor
> changing it since you've already done all the hard work of rolling a
> new stat structure.  I'd rather err on wasting 32-bits for dev_t than
> having to do all this over again.
>    
I would also like to see dev_t changed to 64-bits as part of these
'stat' changes, if that won't cause too much work.  It should be
less work to do it as part of these changes than to do it later!
If changing dev_t to 64-bits turns out to be too much work right
now, then maybe we could at least reserve a 64-bit space and have
the current dev_t be the low-order 32-bits of that space.

-- 
Garance Alistair Drosehn            =   gad at gilead.netel.rpi.edu
Senior Systems Programmer           or  gad at freebsd.org
Rensselaer Polytechnic Institute    or  drosih at rpi.edu



More information about the svn-src-all mailing list