[Pvfs2-developers] Does CVS work rt now?

Sam Lang slang at mcs.anl.gov
Wed Jun 14 15:04:54 EDT 2006


On Jun 12, 2006, at 8:45 PM, Sam Lang wrote:

>
> On Jun 12, 2006, at 3:55 PM, Bradley W Settlemyer wrote:
>
>> Is there a strong reason not to #define these strings somewhere?
>>
>
> I don't think so.  Making them #defines would mean either replacing  
> the strings in the array with the defs, or creating the structs  
> dynamically.  I think sizeof on an inlined string gives the length  
> of the string, so we might be able to do something like:
>
> #define METAFILE_DIST "md"
>
> Trove_Common_Keys[] =
> {
> 	{METAFILE_DIST, sizeof(METAFILE_DIST)},
> 	...
> }
>

I went ahead and committed these changes, hopefully they will prevent  
similar bugs in the future.  The #define strings are located in pvfs2- 
internal.h.

Also, I removed the lines from genconfig that output the  
AttrCacheKeywords options to the config file.  The default is already  
set to the values that were being output, and I don't see these  
options as something most users would want to change (or something we  
would want them to know about).  The same with AttrCacheSize and  
AttrCacheMaxNumElems.  Users can still add non-default values to the  
config file if they wish.

-sam

> BTW, the reason I changed all the strings to two characters:   
> Julian found that it would reduce the size of the keyval berkeley  
> db on disk by about 10%.
>
> -sam
>
>> Cheers,
>> Brad
>>
>>
>> Sam Lang wrote:
>>> I missed a keyval string in mkspace.c.  I've committed a fix  
>>> which  seems to allow ls to work properly now.
>>> -sam
>>> On Jun 12, 2006, at 3:26 PM, Sam Lang wrote:
>>>>
>>>> Hmm...doesn't work for me either.  I'll keep debugging.
>>>>
>>>> -sam
>>>>
>>>> On Jun 12, 2006, at 3:19 PM, Bradley W Settlemyer wrote:
>>>>
>>>>> It no longer core dumps when I do a pvfs2-ls on a freshly  
>>>>> created  filesystem, but I still cannot get pvfs2-ls to  
>>>>> function on a just  created file system.
>>>>>
>>>>> Is it me or you?
>>>>>
>>>>> Cheers,
>>>>> brad
>>>>>
>>>>>
>>>>> Sam Lang wrote:
>>>>>
>>>>>> Brad,
>>>>>> I think this was caused by a commit that I made last week to  
>>>>>> set   shorten the keyval strings.  I had the wrong values for  
>>>>>> the  lengths.   I've committed a fix, can you update and try  
>>>>>> again?
>>>>>> -sam
>>>>>> On Jun 12, 2006, at 2:38 PM, Bradley W Settlemyer wrote:
>>>>>>
>>>>>>> I'm not certain whether its something I've done or what, but   
>>>>>>> it's  pretty screwed up.  I just did a pvfs2-ls, and got this:
>>>>>>>
>>>>>>> software/pvfs2/bin/pvfs2-ls /parl/bradles/software/pvfs2/mnt/
>>>>>>> PVFS_sys_readdir: Invalid argument
>>>>>>>
>>>>>>> My attributes are totally hosed, here is the server:
>>>>>>>
>>>>>>> [D 06/12 15:34] (0x929a468) getconfig (prelude sm) state:  
>>>>>>> req_sched
>>>>>>> [D 06/12 15:34] (0x929a468) getconfig (prelude sm) state:    
>>>>>>> getattr_if_needed
>>>>>>> [D 06/12 15:34] (0x929a468) getconfig (prelude sm) state:    
>>>>>>> perm_check (status = 0)
>>>>>>> [D 06/12 15:34] (0x929a468) getconfig (FR sm) state:  
>>>>>>> release:   (error_code = 0)
>>>>>>> [D 06/12 15:34] (0x929a468) getconfig (FR sm) state:  
>>>>>>> send_resp   (status = 0)
>>>>>>> [D 06/12 15:34] (0x929a468) getconfig (FR sm) state: cleanup
>>>>>>> [D 06/12 15:34] (0x929b498) getattr (prelude sm) state:  
>>>>>>> req_sched
>>>>>>> [D 06/12 15:34] (0x929b498) getattr (prelude sm) state:    
>>>>>>> getattr_if_needed
>>>>>>> [D 06/12 15:34] About to retrieve attributes for handle 1048576
>>>>>>> [D 06/12 15:34] (0x929b498) getattr (prelude sm) state:   
>>>>>>> perm_check  (status = 0)
>>>>>>> [D 06/12 15:34] (0x929b498) getattr (FR sm) state: release:    
>>>>>>> (error_code = -1073742095)
>>>>>>> [D 06/12 15:34] (0x929b498) getattr (FR sm) state:  
>>>>>>> send_resp   (status = -1073741839)
>>>>>>> [D 06/12 15:34] (0x929b498) getattr (FR sm) state: cleanup
>>>>>>> [D 06/12 15:34] (0x929c4c8) getattr (prelude sm) state:  
>>>>>>> req_sched
>>>>>>> [D 06/12 15:34] (0x929c4c8) getattr (prelude sm) state:    
>>>>>>> getattr_if_needed
>>>>>>> [D 06/12 15:34] About to retrieve attributes for handle 1048576
>>>>>>> [D 06/12 15:34] (0x929c4c8) getattr (prelude sm) state:   
>>>>>>> perm_check  (status = 0)
>>>>>>> [D 06/12 15:34] (0x929c4c8) getattr (FR sm) state: release:    
>>>>>>> (error_code = -1073742095)
>>>>>>> [D 06/12 15:34] (0x929c4c8) getattr (FR sm) state:  
>>>>>>> send_resp   (status = -1073741839)
>>>>>>> [D 06/12 15:34] (0x929c4c8) getattr (FR sm) state: cleanup
>>>>>>>
>>>>>>> That release error code is the "invalid argument" error code.
>>>>>>>
>>>>>>> Cheers,
>>>>>>> brad
>>>>>>>
>>>>>>>
>>>>>>> Sam Lang wrote:
>>>>>>>
>>>>>>>> Are you using a previously created storage space?  When was   
>>>>>>>> the  last  time you did a cvs update?  The storage format  
>>>>>>>> has  changed,  so if its  been a while since you updated,  
>>>>>>>> you may  need to run the  migration  tool or re-create your  
>>>>>>>> storage spaces.
>>>>>>>> I can't think of anything else that would bite you...the   
>>>>>>>> nightly   tests seem to have passed last night...
>>>>>>>> -sam
>>>>>>>> On Jun 12, 2006, at 2:03 PM, Bradley W Settlemyer wrote:
>>>>>>>>
>>>>>>>>> I just did a cvs update this morning, not having much luck   
>>>>>>>>> with  the  filesystem.
>>>>>>>>>
>>>>>>>>> Is pvfs2-cp or pvfs2-touch working for anyone else at the    
>>>>>>>>> moment?  Create is returning me an invalid argument problem.
>>>>>>>>>
>>>>>>>>> [D 14:58:55.314670] Handle created: 4294967297
>>>>>>>>> [D 14:58:55.314828] (0x86a2f10) create (FR sm) state:   
>>>>>>>>> release:   (error_code = 0)
>>>>>>>>> [D 14:58:55.314875] (0x86a2f10) create (FR sm) state:   
>>>>>>>>> send_resp   (status = 0)
>>>>>>>>> [D 14:58:55.315046] (0x86a2f10) create (FR sm) state: cleanup
>>>>>>>>> [D 14:58:55.320648] (0x86a3f40) setattr (prelude sm)  
>>>>>>>>> state:   req_sched
>>>>>>>>> [D 14:58:55.320793] (0x86a3f40) setattr (prelude sm)  
>>>>>>>>> state:    getattr_if_needed
>>>>>>>>> [D 14:58:55.320821] About to retrieve attributes for  
>>>>>>>>> handle  1048574
>>>>>>>>> [D 14:58:55.320866] (0x86a3f40) setattr (prelude sm)  
>>>>>>>>> state:    perm_check (status = 0)
>>>>>>>>> [D 14:58:55.320932] (0x86a3f40) setattr (FR sm) state:   
>>>>>>>>> release:   (error_code = -1073742095)
>>>>>>>>> [D 14:58:55.320968] (0x86a3f40) setattr (FR sm) state:   
>>>>>>>>> send_resp   (status = -1073741839)
>>>>>>>>> [D 14:58:55.321064] (0x86a3f40) setattr (FR sm) state: cleanup
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Cheers,
>>>>>>>>> Brad
>>>>>>>>> _______________________________________________
>>>>>>>>> Pvfs2-developers mailing list
>>>>>>>>> Pvfs2-developers at beowulf-underground.org
>>>>>>>>> http://www.beowulf-underground.org/mailman/listinfo/pvfs2-  
>>>>>>>>> developers
>>>>>>>>>
>>>>>>>
>>>>>
>>>>
>>>> _______________________________________________
>>>> Pvfs2-developers mailing list
>>>> Pvfs2-developers at beowulf-underground.org
>>>> http://www.beowulf-underground.org/mailman/listinfo/pvfs2- 
>>>> developers
>>>>
>>
>
> _______________________________________________
> Pvfs2-developers mailing list
> Pvfs2-developers at beowulf-underground.org
> http://www.beowulf-underground.org/mailman/listinfo/pvfs2-developers
>



More information about the Pvfs2-developers mailing list