Re: sync deputy and remote before processing request
"Matt Dew" <[email protected]>
| Newsgroups | gmane.linux.cluster.openmosix.devel |
|---|---|
| Message-ID | <[email protected]> |
Hi Stephan, The effort to help fix openmosix is definitely appreciated. If you're interested in helping with kcomd, grab the patch I posted in my previous email and apply it. Poke around and try to understand what's going on. Feel free to ask questions and I'll try to answer them as best I can. Matt On Tue, 2006-08-01 at 22:34 +0000, Vincent Hanquez wrote: > On Tue, Aug 01, 2006 at 04:49:38PM +0200, Stephan Schmid wrote: > > I made a patch that makes deputy and remote sync if they issue both a > > request. The request of remote is processed first. > > Before returning to usermode remote should process the request of deputy, > > which issues the request again. > > This should happen in remote_pre_usermode, but I think the request hasnt > > yet arrived on remote when it is executed. > > Hi Stephan, > > The whole thing seems to be solving one of the problem that kcomd will > solve. i.e. trying to solve the mess of the current state machine that > is not really complete. > > I think investing effort in kcomd is much better, since it will be able > to provide all the race-protection in a central place and will > simplify a lot some functions. (hopefully) > > > I there a technique to suspend the remote process until something comes > > through the connection ? > > Doing a recv on the socket should do the job. > > apart from that, you can reuse the same om_req structure for a send & > recv. > > and this: > if (err < 0) xxx; > > should be: > if (err < 0) > xxx; > > that isn't *that* bad since it "hide" more the error path, but I'ld > rather have them at a newline. > ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV