in reply to Re^2: Suspend/Resume a process from Tk GUI in Windows XP
in thread Suspend/Resume a process from Tk GUI in Windows XP

Why is it a mystery for you that it is working?

Because you are effectively sharing an object between threads which was never designed to work. This is evidenced by the fact that if you try to make $ProcessObj a shared variable so that you could use it directly from other threads, you get an error message along the lines of:

Invalid value for shared scalar at ...

You've bypassed this design restriction by stringifying the object handle Win32::Process=SCALAR(0x3e820f0) into a shared variable and then string evaling it back into existence in the other thread:$ProcessObj = eval($ProcessObj_as_string).

The problem you had is that this only worked once. What my 'fix' does is avoid it being done multiple times. But the fact that it is not repeatable, means that it works by chance, rather than by design. Indeed, the design was to explicitly prevent it.

In fact the only reason it does work once for this particular object is because the object is a simple blessed scalar reference that contains a process-wide OS handle--which is just a number.

If the object contained any perlish state--that is, any perl instance variables--it wouldn't work.

Basically, you got something to work through sheer luck, but you shouldn't rely upon it for anything remotely serious.


Examine what is said, not who speaks -- Silence betokens consent -- Love the truth but pardon error.
"Science is about questioning the status quo. Questioning authority".
In the absence of evidence, opinion is indistinguishable from prejudice.

Replies are listed 'Best First'.
Re^4: Suspend/Resume a process from Tk GUI in Windows XP
by Dirk80 (Pilgrim) on Feb 18, 2011 at 23:48 UTC

    Thank you again for your answer. That was very interesting.

    But of course I now have the question if there is a way to do it by design and not through sheer luck. Do you know a way?

      But of course I now have the question...

      I thought you might ask, but there's no point in expending energy until required :)

      Instead of passing a handle to the object back to the parent so that it can control it, try giving the parent a shared variable that indicates what it wants the child to do to the object.

      Try this. The thing to look for is $sig, named badly by analogy. Note also the sleep 1. Without it the parent thread will end the process before the child has chance to get a timeslice, with the result that you will create a zombie, which is actually quite hard to do on Win32:

      #!perl -slw use strict; use threads; use threads::shared; use Thread::Queue; use Win32::Process; use Win32; use Data::Dumper; ## A shared var to communicate progress between work thread and TK my $Q = new Thread::Queue; ## A shared var to control the child process. my $sig :shared = ''; sub work{ # save STDOUT open(my $STDOUT_ORIG, ">&", \*STDOUT) or die; # connect TO_CHILD_PROC with CHILD_PROC pipe(CHILD_PROC, TO_CHILD_PROC); # change STDOUT to TO_CHILD_PROC # --> STDOUT of child process is set to TO_CHILD_PROC and # this is connected via pipe with CHILD_PROC open(STDOUT, ">&", \*TO_CHILD_PROC); my $ProcessObj; Win32::Process::Create( $ProcessObj, "C:\\Perl32\\bin\\perl.exe", "perl child.pl", 1, NORMAL_PRIORITY_CLASS, "." )|| die "$! : $^E"; async { { $ProcessObj->Suspend if $sig eq 'Suspend'; $ProcessObj->Resume if $sig eq 'Resume'; $ProcessObj->Kill(-1) and return if $sig eq 'Kill'; Win32::Sleep 100; redo; } }->detach; # restore STDOUT open(STDOUT, ">&", $STDOUT_ORIG) or die; while( <CHILD_PROC> ) { chomp($_); $Q->enqueue( $_ ); } close( CHILD_PROC ); } threads->new( \&work )->detach; use Tk; use Tk::ProgressBar; my $mw = MainWindow->new; my $pb = $mw->ProgressBar()->pack(); my $repeat; $repeat = $mw->repeat( 100 => sub { while( $Q->pending ) { my $progress = $Q->dequeue; return unless $progress; $repeat->cancel if $progress == 100; $pb->value( $progress ); } } ); $mw->Button( '-text' => 'Cancel', '-command' => sub { $sig = 'Kill'; sleep 1; e +xit 0; } )->pack(); $mw->Button( '-text' => 'Suspend', '-command' => sub { $sig = 'Suspend'; } )->pack(); $mw->Button( '-text' => 'Resume', '-command' => sub { $sig = 'Resume'; } )->pack(); $mw->MainLoop;

      Examine what is said, not who speaks -- Silence betokens consent -- Love the truth but pardon error.
      "Science is about questioning the status quo. Questioning authority".
      In the absence of evidence, opinion is indistinguishable from prejudice.