Is my multithreaded form code correct?
Have you considered using BackgroundWorker? It might save you some trouble.

However, it does use ThreadPool and you are better to create your own threads if you have any of the following requirements:


You require a foreground thread.
You require a thread to have a particular priority.
You have tasks that cause the thread to block for long periods of time. The thread pool has a maximum number of threads, so a large number of blocked thread pool threads might prevent tasks from starting.
You need to place threads into a single-threaded apartment. All ThreadPool threads are in the multithreaded apartment.
You need to have a stable identity associated with the thread, or to dedicate a thread to a task.


I add my comment to the answer of "Peter K." here as it seemed to please you.


  Mutex is a bit overkill as it works
  across multiple processes. One should
  use locks (i.e. Monitor) when there is
  no need to synchronize across
  processes. Anyway, I don't see here
  any need for the locks assuming that
  the bDoWork_Click() is always called
  from the UI thread. It only creates a
  new thread if the existing thread is
  not alive. However, the resources used
  by DoWork() must be protected with
  locks if they are shared with other
  threads.