diff options
author | Vladislav Vaintroub <vvaintroub@mysql.com> | 2008-11-14 02:01:41 +0100 |
---|---|---|
committer | Vladislav Vaintroub <vvaintroub@mysql.com> | 2008-11-14 02:01:41 +0100 |
commit | f2a610e1e072f3b50709be3f419ea3ac1d7a3aa5 (patch) | |
tree | dc2b286e5971664d1cad910fc946e7e50a7381b0 /sql/nt_servc.h | |
parent | 60d5e900891bc849d7a6180b708290fe7a3e93ab (diff) | |
download | mariadb-git-f2a610e1e072f3b50709be3f419ea3ac1d7a3aa5.tar.gz |
Bug #20430 mysqld.exe windows service stuck in "SERVICE_STOP_PENDING"
status
The problem appears to be a race condition, when service is being
stopped right after startup. We set the service status to SERVICE_RUNNING
way too early it cannot yet handle stop requests - initialization has
not finished and hEventShutdown that signals server to stop is not yet
created. If somebody issues "net stop MySQL" at this time, MySQL is not
informed about the stop and continues to run as usual, while
NTService::ServiceMain() stucks forever waiting for mysql's "main" thread
to finish.
Solution is to remain in SERVICE_START_PENDING status until after server
initialization is fully complete and only then change the status to
SERVICE_RUNNING. In SERVICE_START_PENDING we do not accept service control
requests, i.e it is not possible to stop service in that time.
sql/mysqld.cc:
Set service status to running after all initialization is complete
sql/nt_servc.cc:
New method SetRunning() to be called by application
to set service status to SERVICE_RUNNING when apllication
has finished initialization.
sql/nt_servc.h:
New method SetRunning() to be called by application
when initialization completes
Diffstat (limited to 'sql/nt_servc.h')
-rw-r--r-- | sql/nt_servc.h | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/sql/nt_servc.h b/sql/nt_servc.h index a3c12569114..9b689e434e1 100644 --- a/sql/nt_servc.h +++ b/sql/nt_servc.h @@ -56,7 +56,19 @@ class NTService BOOL IsService(LPCSTR ServiceName); BOOL got_service_option(char **argv, char *service_option); BOOL is_super_user(); - void Stop(void); //to be called from app. to stop service + + /* + SetRunning() is to be called by the application + when initialization completes and it can accept + stop request + */ + void SetRunning(void); + + /* + Stop() is to be called by the application to stop + the service + */ + void Stop(void); protected: LPSTR ServiceName; |