Hi, > I don't like having a PVFS_hint argument on every sysint call. > Seems like some selectivity would be warranted. What kind of hint > will one ever want to pass to PVFS_mgmt_noop() for instance? > But I'll let others say if they think this is the way to go. Right for the noop maybe, but I was thinking if we add a capability for research purpose then this should be supported by all system calls, you never know what people want to do. For instance new managment functionality or extending the parameter list of a managment function can be experimentally implemented by adding a hint and extracting it later, then you don't have to modify the request protocoll, taking care of the right encode decode etc.. (this is for reseach)
> Weren't you just wanting to generate a custom distribution that > kept your file on a single host? I thought that was possible > already. No, we want to select the nodes on which the datafiles are placed, that is a subset of all servers and not only a single server. We also discussed the request ID (I'm not sure if you were present that time ?), the request ID should be an ID which allows to track which client operation invokes which server operations. This allows a visualisation of client and server I/O activities. I know the hint patch is not completed due to the issue that the hint is not given to the job interface right now, but I will add the cababilities once we aggree on how this should be implemented. > (Looks like some random aio hacking crept into this patch too.) Oh, I see I modified a couple of things in my branch and made a "cvs diff", well just ignore the crap ;-) > Can you use the existing encode_string and decode_string here? Grep > for "string," in src/proto/pvfs2-req-proto.h to see how other > structs use it. Your functions seem to replicate that code. Thanks for pointing that out ! I was not aware that this encode functions exists I > Since you just tacked on after credentials, presumably someone > already thought of the aligment between the unions and the > credentials. Then you're okay as credentials occupy an even 8 bytes. Cool ! Thanks, julian _______________________________________________ Pvfs2-developers mailing list [email protected] http://www.beowulf-underground.org/mailman/listinfo/pvfs2-developers
