April 2002 - Wine-Devel - list.winehq.org

Re: SharedUserData

by Laurent Pinchart

> > >>I need to implement the SharedUserData, which is a page of memory >>located at (KERNEL_BASE - 0x10000) shared between user space (read-only >>access) and kernel space. >> >I don't know how to get the 'page fault' trap for a 'valid' user >page on all systems that wine might run on. Certainly the assumption >that (KERNEL_BASE - 0x10000) is available is probably false... > >However you don't actually need a valid user address, a kernel >address (ie one that is always invalid) will do. > I'm not sure to understand what you mean. The user program will try to access KERNEL_BASE - 0x10000 (which is 0x7ffe0000 on IA-32). I need that to trigger a segfault. >Catching SIGSEGV might be enough - but I'm not totally >certain how you can get the value back into the main >process context (emulate the instruction and modify the >saved program counter ????) > That's what I thought at first. I could also try to set a hardware breakpoint for read/write accesses at that address, but that's highly not portable, and won't work nicely with a debugger. If someone has a better idea I'll be glad to hear it. Laurent Pinchart

April 3, 2002

Re: Removed sorting of registry key values

by Andriy Palamarchuk

--- Eric Pouech <eric.pouech(a)wanadoo.fr> wrote: > I fail to see the benefits of the patch: > - key are sorted to allow O(log(n)) searching > (registry > is loaded only once, and then searched lots of > time) > so using enhanced searching techniques (<< O(n)) > is > really needed > - saying that NT doesn't export the keys in an > alphabetical order doesn't mean the keys are not > sorted. So far, you can only tell that they are > not > sorted, in the export mechanism, by an > alphabetical > order. They could be sorted, for example, by a > hash > code value (look at misc/registry.c for some more > details on registry internals) The tests show that values are exported in the same order in which they are created. This means the values are stored not in any particular order. Sure, Windows internallty can use map of keys for faster search but I doubt it. A program can expect to retrieve values in the order they were created. In particular, the problem can be if the app creates names like Item1, ..., Item10. After sorting Item10 will be after Item1, not after Item9. I don't have any doubts that existing code is faster. The point of the patch is compatibility with Windows behaviour. Andriy Palamarchuk __________________________________________________ Do You Yahoo!? Yahoo! Tax Center - online filing with TurboTax http://taxes.yahoo.com/

April 3, 2002

Re: Removed sorting of registry key values

by Eric Pouech

Andriy Palamarchuk a écrit : > > Working on regedit replacement I noticed that Windows > regedit does not sort registry key values, while Wine > does. > I tested this on NT 4.0, SP2. You can check this by > importing a registry file with not alphabetic values > order and exporting it again. I fail to see the benefits of the patch: - key are sorted to allow O(log(n)) searching (registry is loaded only once, and then searched lots of time) so using enhanced searching techniques (<< O(n)) is really needed - saying that NT doesn't export the keys in an alphabetical order doesn't mean the keys are not sorted. So far, you can only tell that they are not sorted, in the export mechanism, by an alphabetical order. They could be sorted, for example, by a hash code value (look at misc/registry.c for some more details on registry internals) A+

April 3, 2002

DirectX API information - where?

by Tels

-----BEGIN PGP SIGNED MESSAGE----- Moin, I want to learn more about the directx and general windows APIs. Now I looked at some books, but was very unsure as to buy what and where, if ever. First, it seems that good books about these topics come from Microsoft Press. Naturally I don't want to support them ;) Second, it seems that you need for the examples and SDK Visual C++ - again from MS. Eeek! And of course, you have to run and use it under Windows *sigh* I am also worried about the different EULAs, licence and all that other stuff "tainting" me - even though I do believe that all of this stuff isn't legal binding for me anyway. But you never know. So, my question: Where can I obtain *free* information about the API's, their working etc. What do you use? Books you can advise? Thank you in advance, Tels - -- "Why do you go so slowly? Do you think this is some kind of game?" PGP key available on http://bloodgate.com/tels.asc or via email. perl -MDev::Bollocks -e'print Dev::Bollocks->rand(),"\n"' appropriately cultivate error-free convergence -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.0.6 (GNU/Linux) Comment: When cryptography is outlawed, bayl bhgynjf jvyy unir cevinpl. iQEVAwUBPKs2zXcLPEOTuEwVAQGhjQf+KlvVhQQwEOpoReUTiukkohRolyyR8Z+f I/G4sx/ER8+RxtnwrWjBl/gadtnmUdjcf5x0Maca0cxRESjUwSvLgFXNA/VqGtiu SDsDpGVgXUr3iGDu2vXPMFCP0EqrwIUsFSoCpk9me6lWoPlWIOG/EZknA+JY+2zp axFcdrcu7QXq5KF2imPu+ACTA4dC9Af7HGhRflL0ERSe8a4+yhFqC8R2HV++TLPU +H5BQzLPiOANdrfj63bdaCoMWB+qj9bRW+PhPGkzq/QL4p/wsnz70AuW60pSRW5R dOokojLj9gjTY4QEQ/UEt8kyGIxFnSCF21oz9Ltjn1MXC6L8cwJ0Fg== =c8ZH -----END PGP SIGNATURE-----

April 3, 2002

Re: Regression: Wine won't compile with latest cvs shlwapi

by Tony Lambregts

Alexandre Julliard wrote: >Tony Lambregts <tony_lambregts(a)telusplanet.net> writes: > >>I can't get recent cvs versions to compile. After running cvs update -PAd and >>./tools/installwine I get these messages gcc: Internal compiler error: >>program cc1 got fatal signal 11 >> >>make[2]: *** [ordinal.o] Error 1 >>make[2]: Leaving directory `/usr/local/src/wine/dlls/shlwapi' >>make[1]: *** [shlwapi/shlwapi.dll.so] Error 2 >>make[1]: Leaving directory `/usr/local/src/wine/dlls' >>make: *** [dlls] Error 2 >> > >That's most likely the usual gcc bug with local WINAPI function >pointers. I've merged a patch to shlwapi that should work around >this. > Your right my version of gcc is buggy . Thankyou for the patch wine compiles fine now. Tony Lambregts

April 3, 2002

process detach in user

by David Hammerton

Hi, If you take a look at the bottom of dlls/user/user_main.c you will notice that in UserClientDllInitialize we have a switch for 'reason': switch(reason) { case DLL_PROCESS_ATTACH: ret = process_attach(); break; case DLL_THREAD_DETACH: thread_detach(); break; } Should we not also be accounting for the DLL_PROCESS_DETACH case in this switch? The reason I noticed this is because some windows (on the user side) are not being cleaned up when a when a process detaches, although they are being clean up on the x11drv side. Is this just a silly bug or is there a reason DLL_PROCESS_DETACH is not worried about? Cheers David -- David Hammerton programmer and support TransGaming Technologies Inc. http://www.transgaming.com david(a)transgaming.com

April 3, 2002

Re: SharedUserData

by David Laight

On Wed, Apr 03, 2002 at 09:49:07AM +0200, Laurent Pinchart wrote: > Hi everybody, > > I need to implement the SharedUserData, which is a page of memory > located at (KERNEL_BASE - 0x10000) shared between user space (read-only > access) and kernel space. I don't know how to get the 'page fault' trap for a 'valid' user page on all systems that wine might run on. Certainly the assumption that (KERNEL_BASE - 0x10000) is available is probably false... However you don't actually need a valid user address, a kernel address (ie one that is always invalid) will do. Catching SIGSEGV might be enough - but I'm not totally certain how you can get the value back into the main process context (emulate the instruction and modify the saved program counter ????) David -- David Laight: david(a)l8s.co.uk

April 3, 2002

Generalization of async IO structures - New attempt

by Martin Wilck

PATCH: async-struct.diff This patch introduces a new API for asynchronous requests, which 1) separates cleanly between async scheduling and file IO related issues, 2) thereby makes the API compatible with other types of async requests, 3) makes all async IO handling functions static for better DLL separation. This a a reworked version of my patch as of 2002/01/15, with modifications to achieve 3). Patch against: Wine CVS 2002/04/03 Test status: Compiles (no errors/warnings), tested regular file async IO (ok). Added files: include : async.h Modified files: server : protocol.def async.c dlls/kernel : comm.c files : file.c include : file.h scheduler : synchro.c Log message: Martin Wilck <Martin.Wilck(a)Fujitsu-Siemens.com> - new server call: cancel_async - move async IO API from include/file.h, scheduler/synchro.c to new header file include/async.h - differentiate generic async API from IO-type specific fields diff -ruNX diffignore CVS/wine/dlls/kernel/comm.c MW/wine/dlls/kernel/comm.c --- CVS/wine/dlls/kernel/comm.c Tue Apr 2 15:39:18 2002 +++ MW/wine/dlls/kernel/comm.c Wed Apr 3 14:48:39 2002 @@ -91,6 +91,53 @@ WINE_DEFAULT_DEBUG_CHANNEL(comm); +/*********************************************************************** + * Asynchronous I/O for asynchronous wait requests * + */ +#include "async.h" + +static DWORD commio_get_async_status (const async_private *ovp); +static DWORD commio_get_async_count (const async_private *ovp); +static void commio_set_async_status (async_private *ovp, const DWORD status); +static void CALLBACK commio_call_completion_func (ULONG_PTR data); + +static async_ops commio_async_ops = +{ + commio_get_async_status, /* get_status */ + commio_set_async_status, /* set_status */ + commio_get_async_count, /* get_count */ + commio_call_completion_func /* call_completion */ +}; + +typedef struct async_commio +{ + struct async_private async; + LPOVERLAPPED lpOverlapped; + char *buffer; +} async_commio; + +static DWORD commio_get_async_status (const struct async_private *ovp) +{ + return ((async_commio*) ovp)->lpOverlapped->Internal; +} + +static void commio_set_async_status (async_private *ovp, const DWORD status) +{ + ((async_commio*) ovp)->lpOverlapped->Internal = status; +} + +static DWORD commio_get_async_count (const struct async_private *ovp) +{ + return 0; +} + +static void CALLBACK commio_call_completion_func (ULONG_PTR data) +{ + HeapFree(GetProcessHeap(), 0, (void*) data); +} + +/***********************************************************************/ + #if !defined(TIOCINQ) && defined(FIONREAD) #define TIOCINQ FIONREAD #endif @@ -1558,12 +1605,13 @@ */ static void COMM_WaitCommEventService(async_private *ovp) { - LPOVERLAPPED lpOverlapped = ovp->lpOverlapped; + async_commio *commio = (async_commio*) ovp; + LPOVERLAPPED lpOverlapped = commio->lpOverlapped; TRACE("overlapped %p\n",lpOverlapped); /* FIXME: detect other events */ - *ovp->buffer = EV_RXCHAR; + *commio->buffer = EV_RXCHAR; lpOverlapped->Internal = STATUS_SUCCESS; } @@ -1579,8 +1627,8 @@ LPDWORD lpdwEvents, /* [out] event(s) that were detected */ LPOVERLAPPED lpOverlapped) /* [in/out] for Asynchronous waiting */ { - int fd,ret; - async_private *ovp; + int fd; + async_commio *ovp; if(!lpOverlapped) { @@ -1591,53 +1639,32 @@ if(NtResetEvent(lpOverlapped->hEvent,NULL)) return FALSE; - lpOverlapped->Internal = STATUS_PENDING; - lpOverlapped->InternalHigh = 0; - lpOverlapped->Offset = 0; - lpOverlapped->OffsetHigh = 0; - fd = FILE_GetUnixHandle( hFile, GENERIC_WRITE ); if(fd<0) return FALSE; - ovp = (async_private *) HeapAlloc(GetProcessHeap(), 0, sizeof (async_private)); + ovp = (async_commio*) HeapAlloc(GetProcessHeap(), 0, sizeof (async_commio)); if(!ovp) { close(fd); return FALSE; } - ovp->event = lpOverlapped->hEvent; + + ovp->async.ops = &commio_async_ops; + ovp->async.handle = hFile; + ovp->async.fd = fd; + ovp->async.type = ASYNC_TYPE_WAIT; + ovp->async.func = COMM_WaitCommEventService; + ovp->async.event = lpOverlapped->hEvent; ovp->lpOverlapped = lpOverlapped; - ovp->func = COMM_WaitCommEventService; ovp->buffer = (char *)lpdwEvents; - ovp->fd = fd; - ovp->count = 0; - ovp->completion_func = 0; - ovp->type = ASYNC_TYPE_WAIT; - ovp->handle = hFile; - - ovp->next = NtCurrentTeb()->pending_list; - ovp->prev = NULL; - if(ovp->next) - ovp->next->prev=ovp; - NtCurrentTeb()->pending_list = ovp; - - /* start an ASYNCHRONOUS WaitCommEvent */ - SERVER_START_REQ( register_async ) - { - req->handle = hFile; - req->overlapped = lpOverlapped; - req->type = ASYNC_TYPE_WAIT; - req->count = 0; - req->func = check_async_list; - req->status = STATUS_PENDING; - ret=wine_server_call_err(req); - } - SERVER_END_REQ; + lpOverlapped->InternalHigh = 0; + lpOverlapped->Offset = 0; + lpOverlapped->OffsetHigh = 0; - if (!ret) - SetLastError(ERROR_IO_PENDING); + if ( !register_new_async (&ovp->async) ) + SetLastError( ERROR_IO_PENDING ); return FALSE; } diff -ruNX diffignore CVS/wine/files/file.c MW/wine/files/file.c --- CVS/wine/files/file.c Tue Apr 2 15:39:31 2002 +++ MW/wine/files/file.c Wed Apr 3 16:00:38 2002 @@ -64,6 +64,65 @@ WINE_DEFAULT_DEBUG_CHANNEL(file); +/*********************************************************************** + * Asynchronous file I/O * + */ +#include "async.h" + +static DWORD fileio_get_async_status (const async_private *ovp); +static DWORD fileio_get_async_count (const async_private *ovp); +static void fileio_set_async_status (async_private *ovp, const DWORD status); +static void CALLBACK fileio_call_completion_func (ULONG_PTR data); + +static async_ops fileio_async_ops = +{ + fileio_get_async_status, /* get_status */ + fileio_set_async_status, /* set_status */ + fileio_get_async_count, /* get_count */ + fileio_call_completion_func /* call_completion */ +}; + +typedef struct async_fileio +{ + struct async_private async; + LPOVERLAPPED lpOverlapped; + LPOVERLAPPED_COMPLETION_ROUTINE completion_func; + char *buffer; + int count; +} async_fileio; + +static DWORD fileio_get_async_status (const struct async_private *ovp) +{ + return ((async_fileio*) ovp)->lpOverlapped->Internal; +} + +static void fileio_set_async_status (async_private *ovp, const DWORD status) +{ + ((async_fileio*) ovp)->lpOverlapped->Internal = status; +} + +static DWORD fileio_get_async_count (const struct async_private *ovp) +{ + async_fileio *fileio = (async_fileio*) ovp; + DWORD ret = fileio->count - fileio->lpOverlapped->InternalHigh; + return (ret < 0 ? 0 : ret); +} + +static void CALLBACK fileio_call_completion_func (ULONG_PTR data) +{ + async_fileio *ovp = (async_fileio*) data; + TRACE ("data: %p\n", ovp); + + if (ovp->completion_func) + ovp->completion_func(ovp->lpOverlapped->Internal, + ovp->lpOverlapped->InternalHigh, + ovp->lpOverlapped); + + HeapFree(GetProcessHeap(), 0, ovp); +} + +/***********************************************************************/ + #if defined(MAP_ANONYMOUS) && !defined(MAP_ANON) #define MAP_ANON MAP_ANONYMOUS #endif @@ -1344,30 +1403,6 @@ return (r==WAIT_OBJECT_0); } - -/*********************************************************************** - * FILE_StartAsync (INTERNAL) - * - * type==ASYNC_TYPE_NONE means cancel the indicated overlapped operation - * lpOverlapped==NULL means all overlappeds match - */ -BOOL FILE_StartAsync(HANDLE hFile, LPOVERLAPPED lpOverlapped, DWORD type, DWORD count, DWORD status) -{ - BOOL ret; - SERVER_START_REQ(register_async) - { - req->handle = hFile; - req->overlapped = lpOverlapped; - req->type = type; - req->count = count; - req->func = check_async_list; - req->status = status; - ret = wine_server_call( req ); - } - SERVER_END_REQ; - return !ret; -} - /*********************************************************************** * CancelIo (KERNEL32.@) */ @@ -1377,16 +1412,25 @@ TRACE("handle = %x\n",handle); - ovp = NtCurrentTeb()->pending_list; - while(ovp) + for (ovp = NtCurrentTeb()->pending_list; ovp; ovp = t) { t = ovp->next; - if(FILE_StartAsync(handle, ovp->lpOverlapped, ovp->type, 0, STATUS_CANCELLED)) + if (ovp->handle != handle) continue; + + SERVER_START_REQ ( cancel_async ) { - TRACE("overlapped = %p\n",ovp->lpOverlapped); - finish_async(ovp, STATUS_CANCELLED); + req->handle = handle; + req->type = ovp->type; + req->overlapped = ovp; + wine_server_call ( req ); + if (reply->cancelled) + { + TRACE ("cancelling request: %p\n", ovp); + ovp->ops->set_status ( ovp, STATUS_CANCELLED ); + finish_async (ovp); + } } - ovp = t; + SERVER_END_REQ; } WaitForMultipleObjectsEx(0,NULL,FALSE,1,TRUE); return TRUE; @@ -1400,18 +1444,19 @@ */ static void FILE_AsyncReadService(async_private *ovp) { - LPOVERLAPPED lpOverlapped = ovp->lpOverlapped; + async_fileio *fileio = (async_fileio*) ovp; + LPOVERLAPPED lpOverlapped = fileio->lpOverlapped; int result, r; int already = lpOverlapped->InternalHigh; - TRACE("%p %p\n", lpOverlapped, ovp->buffer ); + TRACE("%p %p\n", lpOverlapped, fileio->buffer ); /* check to see if the data is ready (non-blocking) */ - result = pread (ovp->fd, &ovp->buffer[already], ovp->count - already, + result = pread (ovp->fd, &fileio->buffer[already], fileio->count - already, OVERLAPPED_OFFSET (lpOverlapped) + already); if ((result < 0) && (errno == ESPIPE)) - result = read (ovp->fd, &ovp->buffer[already], ovp->count - already); + result = read (ovp->fd, &fileio->buffer[already], fileio->count - already); if ( (result<0) && ((errno == EAGAIN) || (errno == EINTR))) { @@ -1429,9 +1474,9 @@ } lpOverlapped->InternalHigh += result; - TRACE("read %d more bytes %ld/%d so far\n",result,lpOverlapped->InternalHigh,ovp->count); + TRACE("read %d more bytes %ld/%d so far\n",result,lpOverlapped->InternalHigh,fileio->count); - if(lpOverlapped->InternalHigh < ovp->count) + if(lpOverlapped->InternalHigh < fileio->count) r = STATUS_PENDING; else r = STATUS_SUCCESS; @@ -1448,7 +1493,7 @@ LPOVERLAPPED_COMPLETION_ROUTINE lpCompletionRoutine, HANDLE hEvent) { - async_private *ovp; + async_fileio *ovp; int fd; TRACE("file %d to buf %p num %ld %p func %p\n", @@ -1468,7 +1513,7 @@ return FALSE; } - ovp = (async_private *) HeapAlloc(GetProcessHeap(), 0, sizeof (async_private)); + ovp = (async_fileio*) HeapAlloc(GetProcessHeap(), 0, sizeof (async_fileio)); if(!ovp) { TRACE("HeapAlloc Failed\n"); @@ -1476,31 +1521,19 @@ close(fd); return FALSE; } - ovp->event = hEvent; + + ovp->async.ops = &fileio_async_ops; + ovp->async.handle = hFile; + ovp->async.fd = fd; + ovp->async.type = ASYNC_TYPE_READ; + ovp->async.func = FILE_AsyncReadService; + ovp->async.event = hEvent; ovp->lpOverlapped = overlapped; ovp->count = bytesToRead; ovp->completion_func = lpCompletionRoutine; - ovp->func = FILE_AsyncReadService; ovp->buffer = buffer; - ovp->fd = fd; - ovp->type = ASYNC_TYPE_READ; - ovp->handle = hFile; - - /* hook this overlap into the pending async operation list */ - ovp->next = NtCurrentTeb()->pending_list; - ovp->prev = NULL; - if(ovp->next) - ovp->next->prev = ovp; - NtCurrentTeb()->pending_list = ovp; - - if ( !FILE_StartAsync(hFile, overlapped, ASYNC_TYPE_READ, bytesToRead, STATUS_PENDING) ) - { - /* FIXME: remove async_private and release memory */ - ERR("FILE_StartAsync failed\n"); - return FALSE; - } - return TRUE; + return !register_new_async (&ovp->async); } /*********************************************************************** @@ -1510,7 +1543,6 @@ LPOVERLAPPED overlapped, LPOVERLAPPED_COMPLETION_ROUTINE lpCompletionRoutine) { - overlapped->Internal = STATUS_PENDING; overlapped->InternalHigh = 0; return FILE_ReadFileEx(hFile,buffer,bytesToRead,overlapped,lpCompletionRoutine, INVALID_HANDLE_VALUE); } @@ -1588,7 +1620,6 @@ } /* at last resort, do an overlapped read */ - overlapped->Internal = STATUS_PENDING; overlapped->InternalHigh = result; if(!FILE_ReadFileEx(hFile, buffer, bytesToRead, overlapped, NULL, overlapped->hEvent)) @@ -1646,18 +1677,19 @@ */ static void FILE_AsyncWriteService(struct async_private *ovp) { - LPOVERLAPPED lpOverlapped = ovp->lpOverlapped; + async_fileio *fileio = (async_fileio *) ovp; + LPOVERLAPPED lpOverlapped = fileio->lpOverlapped; int result, r; int already = lpOverlapped->InternalHigh; - TRACE("(%p %p)\n",lpOverlapped,ovp->buffer); + TRACE("(%p %p)\n",lpOverlapped,fileio->buffer); /* write some data (non-blocking) */ - result = pwrite(ovp->fd, &ovp->buffer[already], ovp->count - already, + result = pwrite(ovp->fd, &fileio->buffer[already], fileio->count - already, OVERLAPPED_OFFSET (lpOverlapped) + already); if ((result < 0) && (errno == ESPIPE)) - result = write(ovp->fd, &ovp->buffer[already], ovp->count - already); + result = write(ovp->fd, &fileio->buffer[already], fileio->count - already); if ( (result<0) && ((errno == EAGAIN) || (errno == EINTR))) { @@ -1674,9 +1706,9 @@ lpOverlapped->InternalHigh += result; - TRACE("wrote %d more bytes %ld/%d so far\n",result,lpOverlapped->InternalHigh,ovp->count); + TRACE("wrote %d more bytes %ld/%d so far\n",result,lpOverlapped->InternalHigh,fileio->count); - if(lpOverlapped->InternalHigh < ovp->count) + if(lpOverlapped->InternalHigh < fileio->count) r = STATUS_PENDING; else r = STATUS_SUCCESS; @@ -1693,7 +1725,8 @@ LPOVERLAPPED_COMPLETION_ROUTINE lpCompletionRoutine, HANDLE hEvent) { - async_private *ovp; + async_fileio *ovp; + int fd; TRACE("file %d to buf %p num %ld %p func %p stub\n", hFile, buffer, bytesToWrite, overlapped, lpCompletionRoutine); @@ -1704,46 +1737,34 @@ return FALSE; } - overlapped->Internal = STATUS_PENDING; - overlapped->InternalHigh = 0; - - if (!FILE_StartAsync(hFile, overlapped, ASYNC_TYPE_WRITE, bytesToWrite, STATUS_PENDING )) + fd = FILE_GetUnixHandle( hFile, GENERIC_WRITE ); + if ( fd < 0 ) { - TRACE("FILE_StartAsync failed\n"); + TRACE( "Couldn't get FD\n" ); return FALSE; } - ovp = (async_private*) HeapAlloc(GetProcessHeap(), 0, sizeof (async_private)); + ovp = (async_fileio*) HeapAlloc(GetProcessHeap(), 0, sizeof (async_fileio)); if(!ovp) { TRACE("HeapAlloc Failed\n"); SetLastError(ERROR_NOT_ENOUGH_MEMORY); + close (fd); return FALSE; } + + ovp->async.ops = &fileio_async_ops; + ovp->async.handle = hFile; + ovp->async.fd = FILE_GetUnixHandle( hFile, GENERIC_WRITE ); + ovp->async.type = ASYNC_TYPE_WRITE; + ovp->async.func = FILE_AsyncWriteService; ovp->lpOverlapped = overlapped; - ovp->event = hEvent; - ovp->func = FILE_AsyncWriteService; + ovp->async.event = hEvent; ovp->buffer = (LPVOID) buffer; ovp->count = bytesToWrite; ovp->completion_func = lpCompletionRoutine; - ovp->fd = FILE_GetUnixHandle( hFile, GENERIC_WRITE ); - ovp->type = ASYNC_TYPE_WRITE; - ovp->handle = hFile; - if(ovp->fd <0) - { - HeapFree(GetProcessHeap(), 0, ovp); - return FALSE; - } - - /* hook this overlap into the pending async operation list */ - ovp->next = NtCurrentTeb()->pending_list; - ovp->prev = NULL; - if(ovp->next) - ovp->next->prev = ovp; - NtCurrentTeb()->pending_list = ovp; - - return TRUE; + return !register_new_async (&ovp->async); } /*********************************************************************** @@ -1753,7 +1774,6 @@ LPOVERLAPPED overlapped, LPOVERLAPPED_COMPLETION_ROUTINE lpCompletionRoutine) { - overlapped->Internal = STATUS_PENDING; overlapped->InternalHigh = 0; return FILE_WriteFileEx(hFile, buffer, bytesToWrite, overlapped, lpCompletionRoutine, INVALID_HANDLE_VALUE); diff -ruNX diffignore CVS/wine/include/async.h MW/wine/include/async.h --- CVS/wine/include/async.h Thu Jan 1 01:00:00 1970 +++ MW/wine/include/async.h Wed Apr 3 15:41:18 2002 @@ -0,0 +1,143 @@ +/* + * Structures and static functions for handling asynchronous I/O. + * + * Copyright (C) 2002 Mike McCormack, Martin Wilck + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + */ + +/* + * This file declares static functions. + * It should only be included by those source files that implement async I/O requests. + */ + +#ifndef __WINE_ASYNC_H +#define __WINE_ASYNC_H + +struct async_private; + +typedef void (*async_handler)(struct async_private *ovp); +typedef void CALLBACK (*async_call_completion_func)(ULONG_PTR data); +typedef DWORD (*async_get_status)(const struct async_private *ovp); +typedef DWORD (*async_get_count)(const struct async_private *ovp); +typedef void (*async_set_status)(struct async_private *ovp, const DWORD status); + +typedef struct async_ops +{ + async_get_status get_status; + async_set_status set_status; + async_get_count get_count; + async_call_completion_func call_completion; +} async_ops; + +typedef struct async_private +{ + struct async_ops *ops; + HANDLE handle; + HANDLE event; + int fd; + async_handler func; + int type; + struct async_private *next; + struct async_private *prev; +} async_private; + +/* All functions declared static for Dll separation purposes */ + +static void WINAPI check_async_list(async_private *asp, DWORD status); + +inline static void finish_async( async_private *ovp ) +{ + if(ovp->prev) + ovp->prev->next = ovp->next; + else + NtCurrentTeb()->pending_list = ovp->next; + + if(ovp->next) + ovp->next->prev = ovp->prev; + + ovp->next = ovp->prev = NULL; + + close( ovp->fd ); + if( ovp->event!=INVALID_HANDLE_VALUE ) + NtSetEvent( ovp->event, NULL ); + + QueueUserAPC( ovp->ops->call_completion, GetCurrentThread(), (ULONG_PTR)ovp ); +} + +inline static BOOL __register_async( async_private *ovp, const DWORD status ) +{ + BOOL ret; + + SERVER_START_REQ( register_async ) + { + req->handle = ovp->handle; + req->overlapped = ovp; + req->type = ovp->type; + req->count = ovp->ops->get_count( ovp ); + req->func = check_async_list; + req->status = status; + ret = wine_server_call( req ); + } + SERVER_END_REQ; + + if ( ret ) ovp->ops->set_status ( ovp, GetLastError() ); + if ( ovp->ops->get_status (ovp) != STATUS_PENDING ) + finish_async (ovp); + + return ret; +} + +#define register_old_async(ovp) \ + __register_async (ovp, ovp->ops->get_status( ovp )); + +inline static BOOL register_new_async( async_private *ovp ) +{ + const DWORD status = STATUS_PENDING; + + ovp->ops->set_status (ovp, status); + + ovp->next = NtCurrentTeb()->pending_list; + ovp->prev = NULL; + if (ovp->next) ovp->next->prev = ovp; + NtCurrentTeb()->pending_list = ovp; + + return __register_async( ovp, status ); +} + +static void WINAPI check_async_list(async_private *asp, DWORD status) +{ + async_private *ovp; + DWORD ovp_status; + + for( ovp = NtCurrentTeb()->pending_list; ovp && ovp != asp; ovp = ovp->next ); + + if(!ovp) + return; + + if( status != STATUS_ALERTED ) + { + ovp_status = status; + ovp->ops->set_status (ovp, status); + } + else ovp_status = ovp->ops->get_status (ovp); + + if( ovp_status == STATUS_PENDING ) ovp->func( ovp ); /* ovp->func may change status */ + + /* This will destroy all but PENDING requests */ + register_old_async( ovp ); +} + +#endif /* __WINE_ASYNC_H */ diff -ruNX diffignore CVS/wine/include/file.h MW/wine/include/file.h --- CVS/wine/include/file.h Tue Apr 2 15:39:33 2002 +++ MW/wine/include/file.h Wed Apr 3 13:07:35 2002 @@ -46,27 +46,6 @@ int flags; } DOS_DEVICE; -/* overlapped private structure */ -struct async_private; -typedef void (*async_handler)(struct async_private *ovp); -typedef struct async_private -{ - LPOVERLAPPED lpOverlapped; - HANDLE handle; - HANDLE event; - int fd; - char *buffer; - async_handler func; - int count; - int type; - LPOVERLAPPED_COMPLETION_ROUTINE completion_func; - struct async_private *next; - struct async_private *prev; -} async_private; - -extern void WINAPI check_async_list(LPOVERLAPPED ov, DWORD status); -extern void finish_async(struct async_private *ovp, DWORD status); - /* locale-independent case conversion */ inline static char FILE_tolower( char c ) { @@ -99,7 +78,6 @@ DWORD attributes, HANDLE template, BOOL fail_read_only, UINT drive_type ); extern HANDLE FILE_CreateDevice( int client_id, DWORD access, LPSECURITY_ATTRIBUTES sa ); -extern BOOL FILE_StartAsync(HANDLE handle, LPOVERLAPPED lpOverlapped, DWORD type, DWORD count, DWORD status); extern LONG WINAPI WIN16_hread(HFILE16,SEGPTR,LONG); diff -ruNX diffignore CVS/wine/scheduler/synchro.c MW/wine/scheduler/synchro.c --- CVS/wine/scheduler/synchro.c Tue Apr 2 15:39:41 2002 +++ MW/wine/scheduler/synchro.c Wed Apr 3 16:01:36 2002 @@ -50,77 +50,6 @@ } } -static void CALLBACK call_completion_routine(ULONG_PTR data) -{ - async_private* ovp = (async_private*)data; - - ovp->completion_func(ovp->lpOverlapped->Internal, - ovp->lpOverlapped->InternalHigh, - ovp->lpOverlapped); - ovp->completion_func=NULL; - HeapFree(GetProcessHeap(), 0, ovp); -} - -void finish_async(async_private *ovp, DWORD status) -{ - ovp->lpOverlapped->Internal=status; - - /* call ReadFileEx/WriteFileEx's overlapped completion function */ - if(ovp->completion_func) - { - QueueUserAPC(call_completion_routine,GetCurrentThread(),(ULONG_PTR)ovp); - } - - /* remove it from the active list */ - if(ovp->prev) - ovp->prev->next = ovp->next; - else - NtCurrentTeb()->pending_list = ovp->next; - - if(ovp->next) - ovp->next->prev = ovp->prev; - - ovp->next=NULL; - ovp->prev=NULL; - - close(ovp->fd); - if(ovp->event!=INVALID_HANDLE_VALUE) - NtSetEvent(ovp->event,NULL); - if(!ovp->completion_func) HeapFree(GetProcessHeap(), 0, ovp); -} - -/*********************************************************************** - * check_async_list - * - * Process a status event from the server. - */ -void WINAPI check_async_list(LPOVERLAPPED overlapped, DWORD status) -{ - async_private *ovp; - - /* fprintf(stderr,"overlapped %p status %x\n",overlapped,status); */ - - for(ovp = NtCurrentTeb()->pending_list; ovp; ovp = ovp->next) - if(ovp->lpOverlapped == overlapped) - break; - - if(!ovp) - return; - - if(status != STATUS_ALERTED) - ovp->lpOverlapped->Internal = status; - - if(ovp->lpOverlapped->Internal==STATUS_PENDING) - { - ovp->func(ovp); - FILE_StartAsync(ovp->handle, ovp->lpOverlapped, ovp->type, 0, ovp->lpOverlapped->Internal); - } - - if(ovp->lpOverlapped->Internal!=STATUS_PENDING) - finish_async(ovp,ovp->lpOverlapped->Internal); -} - - /*********************************************************************** * wait_reply * diff -ruNX diffignore CVS/wine/server/async.c MW/wine/server/async.c --- CVS/wine/server/async.c Tue Apr 2 15:39:41 2002 +++ MW/wine/server/async.c Tue Apr 2 17:17:00 2002 @@ -190,3 +190,28 @@ release_object(obj); } +DECL_HANDLER(cancel_async) +{ + struct object *obj; + + if (!(obj = get_handle_obj( current->process, req->handle, 0, NULL)) ) + return; + + if(obj->ops->queue_async) + { + struct async_queue *q = obj->ops->queue_async(obj, NULL, req->type, 0); + struct async *async; + + async = find_async(q, current, req->overlapped); + if (async) + { + destroy_async(async); + reply->cancelled = 1; + set_select_events(obj,obj->ops->get_poll_events(obj)); + } + else reply->cancelled = 0; + } + + release_object(obj); +} + diff -ruNX diffignore CVS/wine/server/protocol.def MW/wine/server/protocol.def --- CVS/wine/server/protocol.def Tue Apr 2 15:39:41 2002 +++ MW/wine/server/protocol.def Tue Apr 2 17:17:00 2002 @@ -1615,7 +1615,7 @@ #define SERIALINFO_SET_ERROR 0x04 -/* Create/Destroy an async I/O */ +/* Create / reschedule an async I/O */ @REQ(register_async) handle_t handle; /* handle to comm port, socket or file */ void* func; @@ -1629,6 +1629,14 @@ #define ASYNC_TYPE_WRITE 0x02 #define ASYNC_TYPE_WAIT 0x03 +/* Cancel an async I/O */ +(a)REQ(cancel_async) + handle_t handle; + int type; + void* overlapped; +(a)REPLY + int cancelled; +(a)END /* Create a named pipe */ @REQ(create_named_pipe) diff -ruNX diffignore CVS/wine/server/request.h MW/wine/server/request.h --- CVS/wine/server/request.h Tue Apr 2 15:39:41 2002 +++ MW/wine/server/request.h Wed Apr 3 13:06:02 2002 @@ -228,6 +228,7 @@ DECL_HANDLER(get_serial_info); DECL_HANDLER(set_serial_info); DECL_HANDLER(register_async); +DECL_HANDLER(cancel_async); DECL_HANDLER(create_named_pipe); DECL_HANDLER(open_named_pipe); DECL_HANDLER(connect_named_pipe); @@ -388,6 +389,7 @@ (req_handler)req_get_serial_info, (req_handler)req_set_serial_info, (req_handler)req_register_async, + (req_handler)req_cancel_async, (req_handler)req_create_named_pipe, (req_handler)req_open_named_pipe, (req_handler)req_connect_named_pipe, diff -ruNX diffignore CVS/wine/server/trace.c MW/wine/server/trace.c --- CVS/wine/server/trace.c Tue Apr 2 15:39:42 2002 +++ MW/wine/server/trace.c Wed Apr 3 13:06:02 2002 @@ -1853,6 +1853,18 @@ fprintf( stderr, " status=%08x", req->status ); } +static void dump_cancel_async_request( const struct cancel_async_request *req ) +{ + fprintf( stderr, " handle=%d,", req->handle ); + fprintf( stderr, " type=%d,", req->type ); + fprintf( stderr, " overlapped=%p", req->overlapped ); +} + +static void dump_cancel_async_reply( const struct cancel_async_reply *req ) +{ + fprintf( stderr, " cancelled=%d", req->cancelled ); +} + static void dump_create_named_pipe_request( const struct create_named_pipe_request *req ) { fprintf( stderr, " openmode=%08x,", req->openmode ); @@ -2292,6 +2304,7 @@ (dump_func)dump_get_serial_info_request, (dump_func)dump_set_serial_info_request, (dump_func)dump_register_async_request, + (dump_func)dump_cancel_async_request, (dump_func)dump_create_named_pipe_request, (dump_func)dump_open_named_pipe_request, (dump_func)dump_connect_named_pipe_request, @@ -2449,6 +2462,7 @@ (dump_func)dump_get_serial_info_reply, (dump_func)0, (dump_func)0, + (dump_func)dump_cancel_async_reply, (dump_func)dump_create_named_pipe_reply, (dump_func)dump_open_named_pipe_reply, (dump_func)0, @@ -2606,6 +2620,7 @@ "get_serial_info", "set_serial_info", "register_async", + "cancel_async", "create_named_pipe", "open_named_pipe", "connect_named_pipe",

April 3, 2002

Re: Unicode, i18n support

by Waldek Hebisch

Francois Gouget wrote: > So if I understand correctly, Linux does not provide a uniform > interface to the filesystem. I.e. if I do 'touch ~/foo' where foo > contains weird characters I must make sure these are the right > characters for the codepage used by ~, and then if I do 'touch > /mnt/win98/foo', then I must change 'foo' so that its characters now > match the 1251 codepage, and I may have to rewrite foo yet again for > 'touch /zipdrive/foo'. > > Urgh. This is certainly ugly. I thought that Linux would be taking > UTF-8 or something like it for all filesystems and then do the codepage > conversions itself depending on the underlying filesystem. I thought > that this was the point of having all the codepage information in the > kernel for fat filesystems. Native Unix point of view is that filename is string, and what it means depends on userspace programs (presentation). In Linux one can easily change encoding used for presentation, in Poland typical is ISO8859-2. If user types a name which contains national characters such name is stored verbatim on disk. If retrived later using different encoding it may appear garbled. Typical Linux installation will choose "preffered" encoding and set up things so that encoding works well. In particular codepages in kernel and mount options allows to translate names on fat filesystem form (to) "preffered" encoding. When I wrote about weird setup I mean that technically it is possible to use different encodings in different filesystems, and I can imagine various scenarios that do this (basicaly to work with software that insists on specific encoding). I belive that UTF-8 is the way to go, but it is still the future -- last year trying to make UTF-8 system I found that I need a bunch of programs which cannot work with UTF-8 (they work well with any 8-bit encoding) -- Waldek Hebisch hebisch(a)math.uni.wroc.pl or hebisch(a)hera.math.uni.wroc.pl

April 3, 2002

RE: IE6ALL.zip

by Richard Vout

I could use this also. Is it likely I will be able to get it to work with the windows integrated security? I use a proxy server here which is integrated into a Windows NT Domain. I have managed to get smbclient to map to drives but I can't join the two. Personally I'd rather avoid the native DLL's if possible. Thanks, Richard. -----Original Message----- From: Sylvain Petreolle [mailto:spetreolle(a)yahoo.fr] Sent: 02 April 2002 22:55 To: Lonnie Cumberland; wine-devel(a)winehq.com Subject: Re: IE6ALL.zip For me I installed IE6 with a few native DLLs. But sometimes Web sites are opening into a loop. They just appear as entirely loaded, but continue to download things and end with unknown error as said by IE. Even problems appear in 'About' box... What do you need IE6 for ? --- Lonnie Cumberland <lonnie(a)outstep.com> a écrit : > Hello All, > > I have been trying to get the ie6 setup to run but > keep running into > many functions that do not seem to be implemented > yet. > > Has anyone gotten IE6 installed under wine? If so > then is there some > where that I can just download the installed > directories and unpack > them in my wine-c structure? > > Thanks, > Lonnie > > -- > Lonnie Cumberland > OutStep Technologies Incorporated > EMAIL: Lonnie(a)OutStep.com > : Lonnie_Cumberland(a)yahoo.com > > The Basis Express Virtual Office > & > Data Backup and Recovery Services > > URL: http://www.basis-express.com > > "The Virtual Office without boundaries!!!" > > > > > ___________________________________________________________ Do You Yahoo!? -- Une adresse @yahoo.fr gratuite et en français ! Yahoo! Mail : http://fr.mail.yahoo.com

April 3, 2002

SharedUserData

by Laurent Pinchart

Hi everybody, I need to implement the SharedUserData, which is a page of memory located at (KERNEL_BASE - 0x10000) shared between user space (read-only access) and kernel space. Some information in that page are very accurate (system time in 100ns units for instance), so I don't think that it would be possible to create a shared page at (KERNEL_BASE - 0x10000) and refresh it fast enough. Please correct me if I'm wrong, but I think that the way to go would be to create a copy of that page in memory and initialize it when Wine starts (some information never change, like the system root or version number). When a user space program would try to access that page, the page fault exception handler would return the appropriate data. I'm not very familiar with Wine internals, so I might be wrong, or forget some important issues. Comments are more than welcome :-) I'd also appreciate if someone could give me some hints about the implementation (where I should look for the page fault exception handler, where I should initialize that shared data, where I can find the needed information to initialize those data, ...). Thanks in advance. Laurent Pinchart

April 3, 2002

Re: warn:wave:OSS_OpenDevice Another thread is trying to access audio...

by Eric Pouech

> I forgot to attach the patch so here it is. This patch would be the correct way of fixing the issue. Would you mind trying it? A+ Index: dlls/winmm/wineoss/audio.c =================================================================== RCS file: /home/cvs/cvsroot/wine/wine/dlls/winmm/wineoss/audio.c,v retrieving revision 1.54 diff -u -r1.54 audio.c --- dlls/winmm/wineoss/audio.c 1 Apr 2002 21:02:49 -0000 1.54 +++ dlls/winmm/wineoss/audio.c 3 Apr 2002 06:16:32 -0000 @@ -199,8 +199,9 @@ * open the device for both waveout and wavein streams... * this is hackish, but it's the way OSS interface is done... */ -static int OSS_OpenDevice(unsigned req_access) +static int OSS_OpenDevice(unsigned wDevID, unsigned req_access) { + /* wDevID: is not used yet, we handle only one global device /dev/dsp */ #ifdef USE_FULLDUPLEX /* FIXME: race */ if (OSS_OpenCount == 0) @@ -249,8 +250,9 @@ * * */ -static void OSS_CloseDevice(int fd) +static void OSS_CloseDevice(unsigned wDevID, int fd) { + /* wDevID: is not used yet, we handle only one global device /dev/dsp */ #ifdef USE_FULLDUPLEX if (fd != OSS_OpenFD) FIXME("What the heck????\n"); if (--OSS_OpenCount == 0) @@ -290,7 +292,7 @@ /* FIXME: only one device is supported */ memset(&WOutDev[0].caps, 0, sizeof(WOutDev[0].caps)); - if ((audio = OSS_OpenDevice(O_WRONLY)) == -1) return -1; + if ((audio = OSS_OpenDevice(0, O_WRONLY)) == -1) return -1; ioctl(audio, SNDCTL_DSP_RESET, 0); @@ -370,7 +372,7 @@ !(caps & DSP_CAP_BATCH)) WOutDev[0].caps.dwSupport |= WAVECAPS_DIRECTSOUND; } - OSS_CloseDevice(audio); + OSS_CloseDevice(0, audio); TRACE("out dwFormats = %08lX, dwSupport = %08lX\n", WOutDev[0].caps.dwFormats, WOutDev[0].caps.dwSupport); @@ -385,7 +387,7 @@ memset(&WInDev[0].caps, 0, sizeof(WInDev[0].caps)); - if ((audio = OSS_OpenDevice(O_RDONLY)) == -1) return -1; + if ((audio = OSS_OpenDevice(0, O_RDONLY)) == -1) return -1; ioctl(audio, SNDCTL_DSP_RESET, 0); @@ -451,15 +453,15 @@ WInDev[0].caps.dwFormats |= WAVE_FORMAT_1S16; } } - OSS_CloseDevice(audio); + OSS_CloseDevice(0, audio); TRACE("in dwFormats = %08lX\n", WInDev[0].caps.dwFormats); #ifdef USE_FULLDUPLEX - if ((audio = OSS_OpenDevice(O_RDWR)) == -1) return -1; + if ((audio = OSS_OpenDevice(0, O_RDWR)) == -1) return -1; if (IOCTL(audio, SNDCTL_DSP_GETCAPS, caps) == 0) { OSS_FullDuplex = (caps & DSP_CAP_DUPLEX); } - OSS_CloseDevice(audio); + OSS_CloseDevice(0, audio); #endif return 0; @@ -1069,10 +1091,12 @@ if (access(SOUND_DEV, 0) != 0) return MMSYSERR_NOTENABLED; + if (wwo->unixdev != -1) return MMSYSERR_ALLOCATED; /* we want to be able to mmap() the device, which means it must be opened readable, * otherwise mmap() will fail (at least under Linux) */ - wwo->unixdev = OSS_OpenDevice(((dwFlags & WAVE_DIRECTSOUND) || OSS_FullDuplex) ? - O_RDWR : O_WRONLY); + wwo->unixdev = OSS_OpenDevice(wDevID, + ((dwFlags & WAVE_DIRECTSOUND) || OSS_FullDuplex) ? + O_RDWR : O_WRONLY); if (wwo->unixdev == -1) return MMSYSERR_ALLOCATED; fcntl(wwo->unixdev, F_SETFD, 1); /* set close on exec flag */ @@ -1129,7 +1153,7 @@ /* Read output space info for future reference */ if (ioctl(wwo->unixdev, SNDCTL_DSP_GETOSPACE, &info) < 0) { ERR("IOCTL can't 'SNDCTL_DSP_GETOSPACE' !\n"); - OSS_CloseDevice(wwo->unixdev); + OSS_CloseDevice(wDevID, wwo->unixdev); wwo->unixdev = -1; return MMSYSERR_NOTENABLED; } @@ -1204,7 +1228,7 @@ OSS_DestroyRingMessage(&wwo->msgRing); - OSS_CloseDevice(wwo->unixdev); + OSS_CloseDevice(wDevID, wwo->unixdev); wwo->unixdev = -1; wwo->dwFragmentSize = 0; ret = wodNotifyClient(wwo, WOM_CLOSE, 0L, 0L); @@ -1482,13 +1506,13 @@ { DWORD ret = 1; /* FIXME: For now, only one sound device (SOUND_DEV) is allowed */ - int audio = OSS_OpenDevice(OSS_FullDuplex ? O_RDWR : O_WRONLY); + int audio = OSS_OpenDevice(0, OSS_FullDuplex ? O_RDWR : O_WRONLY); if (audio == -1) { if (errno != EBUSY) ret = 0; } else { - OSS_CloseDevice(audio); + OSS_CloseDevice(0, audio); } return ret; } @@ -2271,7 +2295,8 @@ } wwi = &WInDev[wDevID]; - if ((wwi->unixdev = OSS_OpenDevice(OSS_FullDuplex ? O_RDWR : O_RDONLY)) == -1) + if (wwi->unixdev != -1) return MMSYSERR_ALLOCATED; + if ((wwi->unixdev = OSS_OpenDevice(wDevID, OSS_FullDuplex ? O_RDWR : O_RDONLY)) == -1) return MMSYSERR_ALLOCATED; fcntl(wwi->unixdev, F_SETFD, 1); /* set close on exec flag */ if (wwi->lpQueuePtr) { @@ -2323,7 +2348,7 @@ IOCTL(wwi->unixdev, SNDCTL_DSP_GETBLKSIZE, fragment_size); if (fragment_size == -1) { WARN("IOCTL can't 'SNDCTL_DSP_GETBLKSIZE' !\n"); - OSS_CloseDevice(wwi->unixdev); + OSS_CloseDevice(wDevID, wwi->unixdev); wwi->unixdev = -1; return MMSYSERR_NOTENABLED; } @@ -2366,7 +2391,7 @@ } OSS_AddRingMessage(&wwi->msgRing, WINE_WM_CLOSING, 0, TRUE); - OSS_CloseDevice(wwi->unixdev); + OSS_CloseDevice(wDevID, wwi->unixdev); wwi->unixdev = -1; wwi->dwFragmentSize = 0; OSS_DestroyRingMessage(&wwi->msgRing);

April 3, 2002

Re: wine/debugger/stabs.c patch breaks FreeBSD

by Eric Pouech

> (Both of these patches work for me, so I don't know which one's > preferrable...) since Alexandre's one is already committed, that's the one (even, if we may use the ELFMAG? constants someday) A+

April 3, 2002

Re: [PATCH] NtQueryInformationProcess & ProcessDebugPort

by Eric Pouech

> > sdk's windbg already work, so why no VC++ debugger ? > Then we'll probably have to implement NtQueryInformationProcess completely > for ProcessDebugPort. Does anyone know what it should return ? The debugger > process handle ? the debugport value is not intented for ring3 use. so any non null value would do when the debugger is present. A+

April 3, 2002

Re: ExitProcess crashes apps

by Jason Phillips

This problem looks related to bug #460, which also has this error message: > err:win:WIN_FindWndPtr window 10021 belongs to other process > err:win:WIN_FindWndPtr window 10021 belongs to other process http://wine.codeweavers.com/bugzilla/show_bug.cgi?id=460 On Wednesday 03 April 2002 01:18 pm, you wrote: > Hi > After just compiling and installing current cvs wine I get an unhandled > exception every time an app closes. > The culprit seems to be somewhere in kernel32.ExitProcess: > Here is the relay output of the uninstaller that comes with wine (last 50 > lines) > > Enrico > farmboy1(a)subdimension.com > > 08070840:Ret x11drv.SelectPen() retval=000000e6 ret=4070c3e1 > 08070840:Call x11drv.SelectBrush(403b44b0,0000075a) ret=4070c3b2 > 08070840:Ret x11drv.SelectBrush() retval=0000075a ret=4070c3b2 > 08070840:Call x11drv.SelectPen(403b44b0,0000008a) ret=4070c3e1 > 08070840:Ret x11drv.SelectPen() retval=0000008a ret=4070c3e1 > 08070840:Call x11drv.SelectBrush(403b44b0,0000006e) ret=4070c3b2 > 08070840:Ret x11drv.SelectBrush() retval=0000006e ret=4070c3b2 > 08070840:Call x11drv.SetDeviceClipping(403b44b0) ret=406fecee > 08070840:Ret x11drv.SetDeviceClipping() retval=00000000 ret=406fecee > 08070840:Call x11drv.SelectBrush(403b44b0,0000006e) ret=4070c3b2 > 08070840:Ret x11drv.SelectBrush() retval=0000006e ret=4070c3b2 > 08070840:Call x11drv.GetDeviceCaps(403b44b0,00000022) ret=407019ed > 08070840:Ret x11drv.GetDeviceCaps() retval=000079f7 ret=407019ed > 08070840:Call x11drv.SelectFont(403b44b0,000000a2) ret=4070c24c > 08070840:Ret x11drv.SelectFont() retval=00000001 ret=4070c24c > 08070840:Call x11drv.SelectPen(403b44b0,0000008a) ret=4070c3e1 > 08070840:Ret x11drv.SelectPen() retval=0000008a ret=4070c3e1 > 08070840:Call x11drv.SetBkColor(403b44b0,00ffffff) ret=40701a8a > 08070840:Ret x11drv.SetBkColor() retval=00ffffff ret=40701a8a > 08070840:Call x11drv.SetTextColor(403b44b0,00000000) ret=40701b3a > 08070840:Ret x11drv.SetTextColor() retval=00000000 ret=40701b3a > 08070840:Ret window proc 0x405e96f4 > (hwnd=00010025,msg=BM_SETSTATE,wp=00000000,lp=00000000) retval=00000000 > 08070840:Call window proc 0x405e96f4 > (hwnd=00010025,msg=WM_CAPTURECHANGED,wp=00000000,lp=00000000) > 08070840:Ret window proc 0x405e96f4 > (hwnd=00010025,msg=WM_CAPTURECHANGED,wp=00000000,lp=00000000) > retval=00000000 08070840:Call window proc 0x4001c7f8 > (hwnd=00010021,msg=WM_COMMAND,wp=00000002,lp=00010025) > 08070840:Call user32.PostQuitMessage(00000000) ret=4001ce19 > 08070840:Ret user32.PostQuitMessage() retval=00000001 ret=4001ce19 > 08070840:Ret window proc 0x4001c7f8 > (hwnd=00010021,msg=WM_COMMAND,wp=00000002,lp=00010025) retval=00000000 > 08070840:Ret window proc 0x405e96f4 > (hwnd=00010025,msg=WM_LBUTTONUP,wp=00000000,lp=000a0057) retval=00000000 > 08070840:Ret user32.DispatchMessageA() retval=00000000 ret=4001c311 > 08070840:Call user32.GetMessageA(40586e18,00000000,00000000,00000000) > ret=4001c31d > 08070840:Call > x11drv.MsgWaitForMultipleObjectsEx(00000000,00000000,00000000,00000000,0000 >0000) ret=405deb9a > 08070840:Ret x11drv.MsgWaitForMultipleObjectsEx() retval=00000102 > ret=405deb9a > 08070840:Ret user32.GetMessageA() retval=00000000 ret=4001c31d > 08070840:Call kernel32.ExitProcess(00000000) ret=4001b07c > 08070840:Call PE DLL (proc=0x405e5140,module=405c5000,type=0,res=0x1) > 08070840:Ret PE DLL (proc=0x405e5140,module=405c5000,type=0,res=0x1) > retval=1 08070840:Call PE DLL > (proc=0x40829860,module=4081e000,type=0,res=0x1) 080a9e40:Ret > x11drv.MsgWaitForMultipleObjectsEx() retval=00000001 ret=405deec9 > 080a9e40:Call > x11drv.MsgWaitForMultipleObjectsEx(00000000,00000000,00000000,00000000,0000 >0000) ret=405deb9a > 080a9e40:Ret x11drv.MsgWaitForMultipleObjectsEx() retval=00000102 > ret=405deb9a > 080a9e40:Call > x11drv.MsgWaitForMultipleObjectsEx(00000001,403b3070,ffffffff,00000000,0000 >0000) ret=405deec9 > 08070840:Ret PE DLL (proc=0x40829860,module=4081e000,type=0,res=0x1) > retval=1 08070840:Call PE DLL > (proc=0x406ee894,module=406e1000,type=0,res=0x1) 08070840:Ret PE DLL > (proc=0x406ee894,module=406e1000,type=0,res=0x1) retval=1 08070840:Call PE > DLL (proc=0x407749e0,module=40764000,type=0,res=0x1) 08070840:Ret PE DLL > (proc=0x407749e0,module=40764000,type=0,res=0x1) retval=1 wine: Unhandled > exception, starting debugger... > err:win:WIN_FindWndPtr window 10021 belongs to other process > err:win:WIN_FindWndPtr window 10021 belongs to other process -- Jason Phillips ZIPPY SAYS: --> I am covered with pure vegetable oil and I am writing a best seller!

April 3, 2002

Re: error: no implementation for WININET.dll.102(InternetGetConnectedState)

by Alexandre Julliard

"Dimitrie O. Paun" <dpaun(a)rogers.com> writes: > That's true, but if we default to builtin, we expose more of _our_ bugs, > and hopefully we get them fixed faster. Not necessarily; many bugs appear only when using a mix of natives and builtins, since this is where the compatibility requirements are the most strict. -- Alexandre Julliard julliard(a)winehq.com

April 3, 2002

Re: Regression: Wine won't compile with latest cvs shlwapi

by Alexandre Julliard

Tony Lambregts <tony_lambregts(a)telusplanet.net> writes: > I can't get recent cvs versions to compile. After running cvs update -PAd and > ./tools/installwine I get these messages gcc: Internal compiler error: > program cc1 got fatal signal 11 > > make[2]: *** [ordinal.o] Error 1 > make[2]: Leaving directory `/usr/local/src/wine/dlls/shlwapi' > make[1]: *** [shlwapi/shlwapi.dll.so] Error 2 > make[1]: Leaving directory `/usr/local/src/wine/dlls' > make: *** [dlls] Error 2 That's most likely the usual gcc bug with local WINAPI function pointers. I've merged a patch to shlwapi that should work around this. -- Alexandre Julliard julliard(a)winehq.com

April 3, 2002

Re: Regression: Wine won't compile with latest cvs shlwapi

by Tony Lambregts

Ulrich Weigand wrote: >Tony Lambregts wrote: > >>gcc: Internal compiler error: program cc1 got fatal signal 11 >> > >>I talked to Lawson on wine-users and he says I should look at >>http://www.bitwizard.nl/sig11/ (This link is about mostly about compiling a >>kernal but it applies to compiling wine as well) >> >>I have looked at it and tried most of the things suggested in it. I still stop >>in exactly the same place. >> >[snip] > >>What do I need to do now? >> > >If you can get cc1 to die with SIGSEGV reproducibly in same place, >then you have most probably found a compiler bug. > >The first question is of course, which version of gcc are you >using? Did you build it yourself or is it the one that came >with your distro (which one?)? What is the output of > gcc -v > rpm -qf `which gcc` > > >Bye, >Ulrich > Redhat 6.0 gcc -v Reading specs from /usr/lib/gcc-lib/i386-redhat-linux/egcs-2.91.66/specs gcc version egcs-2.91.66 19990314/Linux (egcs-1.1.2 release) Tony Lambregts

April 3, 2002

Re: error: no implementation for WININET.dll.102(InternetGetConnectedState)

by Dimitrie O. Paun

On April 2, 2002 09:02 pm, Alexandre Julliard wrote: > That wouldn't help; sometimes builtins work better and sometimes they > don't, it depends on the application and on the Windows version of the > native dlls. There isn't a single configuration that works better in > all cases. That's true, but if we default to builtin, we expose more of _our_ bugs, and hopefully we get them fixed faster. -- Dimi.

April 3, 2002

Re: error: no implementation for WININET.dll.102(InternetGetConnectedState)

by Alexandre Julliard

Sylvain Petreolle <spetreolle(a)yahoo.fr> writes: > we could put ALL emulated dlls in the default config > file with "something" = "builtin, native" > and # the ones that aren't enough implemented/tested. > > Comments ? That wouldn't help; sometimes builtins work better and sometimes they don't, it depends on the application and on the Windows version of the native dlls. There isn't a single configuration that works better in all cases. -- Alexandre Julliard julliard(a)winehq.com

April 3, 2002

Race condition in TemrinateThread

by Geoffrey Hausheer

I've discovered a race condition in TerminateThread. Basically, if a thread is terminated before it has fully started, than the thread leaves a wine process around when the process exits. I wouldn't have figured this out if it weren't for the OpenThread code which was just added (it slowed down my test enough that with '--winver win2000' the thread was able to start, whereas with '--winver win98' it couldn't (due to testing OpenThread before calling TerminateThread when in win2k mode). It should be duplicatable by simply calling thread=CreateThread(); TerminateThread(thread,0); with no wait between them. Anyhow, I really have no idea how to locate this, but I'd be happy to try if someone can give me a hand. Thanks, .Geoff

April 3, 2002

Re: MaskBlt function almost working

by Lonnie Cumberland

Hello All, I just found out that the Maskblt code that I submitted earlier is free to use, but there is still a little problem with it and I am not sure how to fix it. Cheers, Lonnie > -----BEGIN PGP SIGNED MESSAGE----- > > Moin, > > On 02-Apr-02 Tels carved into stone: >> -----BEGIN PGP SIGNED MESSAGE----- >> >> Moin, >> >> On 02-Apr-02 Dimitrie O. Paun carved into stone: >>> On April 2, 2002 12:48 pm, Lonnie Cumberland wrote: >>>> Hello All, >>> [...] >>>> By the way, I found this function at: >>>> >>>> http://www.codeproject.com/bitmap/maskblt.asp >>>> >>>> and modified it to work with wine. >>> >>> What about the license? Is it LGPL compatible? >> >> I have searched the entire site and I couldn't find a single bit >> about the >> licence that submitted (or otherwise available code at that >> site) falls under. It is heavy MS-centric, so I guess it is not >> very /L?GPL/ friendly. >> >> Best bet would probably be to send them an email and ask ;) > > Ah, I overread this in the FAQ: > >>If you post to CodeProject then you retain copyright of your >>article and code. You also give CodeProject permission to use it >>in a fair manner and also permit all developers to freely use the >>code in their own >>applications. >> >>Note that we do not accept time limited or shareware >>contributions: All code that is available from our site must be >>available for use for free. > > IANLSBIPAH (I am no Licence specialist, but I play one at home) > so somebody else could chime in and tell us if that is > "compatible". Also aksing the original author nonetheless because > it is kind and friendly wouldn#t hurt, either. > > Cheers, > > Tels > > - -- > "Why do you go so slowly? Do you think this is some kind of > game?" PGP key available on http://bloodgate.com/tels.asc or via > email. > perl -MDev::Bollocks -e'print Dev::Bollocks->rand(),"\n"' > widespreadedly synthesize second-generation markets > > > -----BEGIN PGP SIGNATURE----- > Version: GnuPG v1.0.6 (GNU/Linux) > Comment: When cryptography is outlawed, bayl bhgynjf jvyy unir > cevinpl. > > iQEVAwUBPKoBpHcLPEOTuEwVAQEhKgf+OyAxJsidtYoY9O+5r7OjYK9RcKFVF8ED > SiwlbJsohA7Ip681sFovCrcixli/11LuImHSbHCBhgunocq/fAmsSUbhR7gYcd0F > irzaDbRFFHvVUadLBdCecDeaI4FBKK22+dW7zl2uiShfQMb+BgkAsWtOmwBBRLlW > 2qaStWbbNXL+XbHal/m9qGlQ+G4zHOpd7mz0li8B/NMWdbmIv5pFYP2Fs+hKE5t4 > 1546AShXiFyj36cdB9iB71BK2nmYtiECB0b4G7HhR8foT7JDfIZUxP+0LrC14pfh > ITOczNEbZzaVs+lIg3tSdM6nymzaCHHIQWsxXMbtgJwbDd4RJcoxFA== > =FMIm > -----END PGP SIGNATURE----- -- Lonnie Cumberland OutStep Technologies Incorporated EMAIL: Lonnie(a)OutStep.com : Lonnie_Cumberland(a)yahoo.com The Basis Express Virtual Office & Data Backup and Recovery Services URL: http://www.basis-express.com "The Virtual Office without boundaries!!!"

April 3, 2002

Re: Regression: Wine won't compile with latest cvs shlwapi

by Ulrich Weigand

Tony Lambregts wrote: > gcc: Internal compiler error: program cc1 got fatal signal 11 > I talked to Lawson on wine-users and he says I should look at > http://www.bitwizard.nl/sig11/ (This link is about mostly about compiling a > kernal but it applies to compiling wine as well) > > I have looked at it and tried most of the things suggested in it. I still stop > in exactly the same place. [snip] > What do I need to do now? If you can get cc1 to die with SIGSEGV reproducibly in same place, then you have most probably found a compiler bug. The first question is of course, which version of gcc are you using? Did you build it yourself or is it the one that came with your distro (which one?)? What is the output of gcc -v rpm -qf `which gcc` Bye, Ulrich -- Dr. Ulrich Weigand weigand(a)informatik.uni-erlangen.de

April 3, 2002

Re: Where is "cprof"

by Jeff Tranter

I can send you a copy of the source code which I backed up before Corel shut down their open source site. Luca Fini wrote: >I've seen that the Wine development team has also developed a profiler for >multi-threaded programs. > >I'd be very interested in using it to profile an application which I'm >developing (with no connection with wine itlself). > >Unfortunately the site which is suppose to be the cprof home is at Corel, >which I understand is no longer in business. > >Where can I found a workable version of cprof? > >Many thanks for any reoply. > > l.f. >

April 2, 2002

Re: Regression: Wine won't compile with latest cvs shlwapi

by Jeremy Newman

No it's not. You just happened to catch it in the 5 seconds it took me to change a symlink. Good timing on your part I would say. :-D On Tue, 2002-04-02 at 16:57, Dimitrie O. Paun wrote: > On April 2, 2002 05:34 pm, Tony Lambregts wrote: > > > This is the patch that was commited at 19:33:39 CST > > http://www.winehq.com/hypermail/wine-cvs/2002/03/0072.html > > Hey!!! wine-cvs archive is down! ;( > > -- > Dimi. > -- __ _WebGeek/NetAdmin CodeWeavers -= http://www.codeweavers.com | \| |_____ __ ___ __ __ _ _ _ -= http://www.dracowulf.com | .` / -_) | | / ' \/ _` | ' \ -= mailto:jnewman(a)codeweavers.com |_|\_\___|\_/\_/|_|_|_\__,_|_||_| -= ICQ: 1842980 Yahoo: laxdragon

April 2, 2002