diff options
| author | Anatol Belski <ab@php.net> | 2015-06-01 21:52:35 +0200 |
|---|---|---|
| committer | Anatol Belski <ab@php.net> | 2015-06-02 15:56:32 +0200 |
| commit | 06332e67067e9fea3117cbce8c164d99027ee142 (patch) | |
| tree | 8c40197f2b2bf74be4fea77d5a58d118d99565e5 /win32 | |
| parent | 29a91e52adfdcd7fa5126e740190451eef98a553 (diff) | |
| download | php-git-06332e67067e9fea3117cbce8c164d99027ee142.tar.gz | |
introduced DllMain for the main php DLL
The particular need on this is because of the current situation
with determining the background functionality for the
gettimeofday. DllMain allows to initialize stuff before the DLL
can be actually used. Thus, we use different time API on win7
and win8 and later, so the function pointer needs to be
initialized before anything in the DLL could even demand it.
The change also opens the door for the further optimizations,
as now we're able to do the very basic initializations for the
whole DLL before it could ever start to live. Fe on this way the
TLS initialization could be done, when utilizing the
DLL_THREAD_ATTACH/DETACH case. Whether it's really usable in
portable way should be synced with other platforms.
Be aware that it's dangerous as it possibly causes dead locks.
So to use with care. One willing to add items to DllMain should
better read the documentation twice and even then try to defer
the necessary action.
Diffstat (limited to 'win32')
| -rw-r--r-- | win32/build/config.w32 | 2 | ||||
| -rw-r--r-- | win32/dllmain.c | 70 | ||||
| -rw-r--r-- | win32/time.c | 14 | ||||
| -rw-r--r-- | win32/time.h | 6 |
4 files changed, 87 insertions, 5 deletions
diff --git a/win32/build/config.w32 b/win32/build/config.w32 index 70aef37799..b7e5e49220 100644 --- a/win32/build/config.w32 +++ b/win32/build/config.w32 @@ -159,7 +159,7 @@ ADD_SOURCES("main/streams", "streams.c cast.c memory.c filter.c plain_wrapper.c userspace.c transports.c xp_socket.c mmap.c glob_wrapper.c"); ADD_FLAG("CFLAGS_BD_MAIN_STREAMS", "/D ZEND_ENABLE_STATIC_TSRMLS_CACHE=1"); -ADD_SOURCES("win32", "glob.c readdir.c \ +ADD_SOURCES("win32", "dllmain.c glob.c readdir.c \ registry.c select.c sendmail.c time.c winutil.c wsyslog.c globals.c getrusage.c"); ADD_FLAG("CFLAGS_BD_WIN32", "/D ZEND_ENABLE_STATIC_TSRMLS_CACHE=1"); diff --git a/win32/dllmain.c b/win32/dllmain.c new file mode 100644 index 0000000000..9e7bcc1c9b --- /dev/null +++ b/win32/dllmain.c @@ -0,0 +1,70 @@ +/* + +----------------------------------------------------------------------+ + | PHP Version 7 | + +----------------------------------------------------------------------+ + | Copyright (c) 1997-2015 The PHP Group | + +----------------------------------------------------------------------+ + | This source file is subject to version 3.01 of the PHP license, | + | that is bundled with this package in the file LICENSE, and is | + | available through the world-wide-web at the following url: | + | http://www.php.net/license/3_01.txt | + | If you did not receive a copy of the PHP license and are unable to | + | obtain it through the world-wide-web, please send a note to | + | license@php.net so we can mail you a copy immediately. | + +----------------------------------------------------------------------+ + | Authors: Anatol Belski <ab@php.net> | + +----------------------------------------------------------------------+ + */ + +#include <config.w32.h> + +#include <win32/time.h> +#include <php.h> + +#ifdef HAVE_LIBXML +#include <libxml/threads.h> +#endif + +/* TODO this file, or part of it, could be machine generated, to + allow extensions and SAPIs adding their own init stuff. + However expected is that MINIT is enough in most cases. + This file is only useful for some really internal stuff, + eq. initializing something before the DLL even is + available to be called. */ + +BOOL WINAPI DllMain(HINSTANCE inst, DWORD reason, LPVOID dummy) +{ + BOOL ret = TRUE; + + switch (reason) + { + case DLL_PROCESS_ATTACH: + ret = ret && php_win32_init_gettimeofday(); + break; +#if 0 /* prepared */ + case DLL_PROCESS_DETACH: + /* pass */ + break; + + case DLL_THREAD_ATTACH: + /* pass */ + break; + + case DLL_THREAD_DETACH: + /* pass */ + break; +#endif + } + +#ifdef HAVE_LIBXML + /* This imply that only LIBXML_STATIC_FOR_DLL is supported ATM. + If that changes, this place will need some rework. + TODO Also this should be revisited as no initialization + might be needed for TS build (libxml build with TLS + support. */ + ret = ret && xmlDllMain(inst, reason, dummy); +#endif + + return ret; +} + diff --git a/win32/time.c b/win32/time.c index f2487af9fc..dfcc46a59b 100644 --- a/win32/time.c +++ b/win32/time.c @@ -29,10 +29,11 @@ typedef VOID (WINAPI *MyGetSystemTimeAsFileTime)(LPFILETIME lpSystemTimeAsFileTi static MyGetSystemTimeAsFileTime timefunc = NULL; +#ifdef PHP_EXPORTS static zend_always_inline MyGetSystemTimeAsFileTime get_time_func(void) { MyGetSystemTimeAsFileTime timefunc = NULL; - HMODULE hMod = LoadLibrary("kernel32.dll"); + HMODULE hMod = GetModuleHandle("kernel32.dll"); if (hMod) { /* Max possible resolution <1us, win8/server2012 */ @@ -47,15 +48,20 @@ static zend_always_inline MyGetSystemTimeAsFileTime get_time_func(void) return timefunc; } +BOOL php_win32_init_gettimeofday(void) +{ + timefunc = get_time_func(); + + return (NULL != timefunc); +} +#endif + static zend_always_inline int getfilesystemtime(struct timeval *tv) { FILETIME ft; unsigned __int64 ff = 0; ULARGE_INTEGER fft; - if (!timefunc) { - timefunc = get_time_func(); - } timefunc(&ft); /* diff --git a/win32/time.h b/win32/time.h index 8f39c0481c..b46c1675ca 100644 --- a/win32/time.h +++ b/win32/time.h @@ -52,4 +52,10 @@ PHPAPI int nanosleep( const struct timespec * rqtp, struct timespec * rmtp ); PHPAPI int usleep(unsigned int useconds); +#ifdef PHP_EXPORTS +/* This symbols are needed only for the DllMain, but should not be exported + or be available when used with PHP binaries. */ +BOOL php_win32_init_gettimeofday(void); +#endif + #endif |
