On Tue, Aug 10, 2004 at 02:11:50PM +0200, Lazar Obradovic wrote: > On Tue, 2004-08-10 at 11:29, Patrick Caulfield wrote: > > OK, I've had to do this slightly differently, using gethostname2_r, it has > > the same effect. It behaves itself on my cluster but let me know if I've missed > > anything. > > It's ok, just that I wanted cleaner solution (cleaner = w/o additional > vars). > > As you might notice, TTL is fixed to a value of 10, and it might be > interesting to take this out of the code, and place it somewhere in > cluster.conf. Do you think this is ok? > > how about something like: > > #define MCAST_TTL_PATH "//cluster/cman/multicast/@ttl" > > Let me know if this is generaly a good idea, I'll work on details if you > do agree. Certainly. I'm all in favour of moving hard-coded values into configuration files - so long as the defaults are reasonable ! > I also started to change ccs a bit for mcast support. It turns out that > ccs has a lot of definitions hardcoded. Can I take 'em out and put into > separate header file (comm_header.h looks nice :)? I think ccs_join.h would be reasonable, then it's obvious which .c file it holds the defaults for. -- patrick
