May 2004 - Wine-Devel - list.winehq.org

Unsupported feature in SHFileOperationW
by Lionel Ulmer May 28, 2004

May 28, 2004

Hi all, A game demo installer uses SHFileOperationA to copy files over... And it expects also the destination directories to be created when the copy is done. Basically, it gives a list of fully qualified source files and their corresponding fully qualified destination files and cann the SHFileOperationA function with the 'FO_COPY' function and 'FOF_MULTIDESTFILES FOF_NOCONFIRMATION FOF_NOCONFIRMMKDIR' flags. This seems to not be handled properly in the current Wine tree as it fails as soon as it needs to create the directory on the destination files. I tested this on Windows and it works just fine (with nice MessageBoxes if one ommits the 'NOCONFIRMMKDIR' flag :-) ). Is anyone (Martin, Ge ?) currently working on Shell32 who has an updated tree with fixes in the Shell file operations ? If not, I will try to fix it on my own (be aware though that as I find the style of coding used in this file particularely hard to decipher, I may do a 'NIH' on it and rewrite most of it to fit my tastes :-) ). Lionel -- Lionel Ulmer - http://www.bbrox.org/

1 0

focused code review
by Glen Kaukola May 28, 2004

May 28, 2004

Hi, So one project suggested on the website is to do a focused code review. In particular it talks about getting rid of inefficient uses of strcat. But that leaves me wondering what exactly constitutes an inefficient use of strcat? If anyone could provide an example I'd appreciate it. Thanks. Glen

2 1

Re: Default to 96dpi for the no config file case
by Rolf Kalbermatter May 28, 2004

May 28, 2004

>If you mean via DDC then I guess this should work in most places, but for >whatever reason it does not work here :( In particular, xdpyinfo says: > >screen #0: > dimensions: 1024x768 pixels (321x241 millimeters) > resolution: 81x81 dots per inch > >So it appears to know the screen size (I measured with a ruler and >321x241 is right), but it gets the DPI too small anyway. At 81 dpi the >text is hardly readable. But: 1024/321*25.4 = 81.026 768/241*25.4 = 80.943 Seems to me X has it very right. That Windows usually doesn't care about real resolutions and instead fudges with it (consider the Large Font setting in the Display Control Panel) is probably another issue. Or the metrics this function requires are not dots per inch but something else. Rolf Kalbermatter

1 0

Re: Default to 96dpi for the no config file case
by Rein Klazes May 28, 2004

May 28, 2004

On Thu, 27 May 2004 22:18:20 +0100, you wrote: > wineprefixcreate doesn't create a config file. This means we use a (wrong) > formula to decide what the DPI is supposed to be. Why do you mean (wrong) ? Do you not get the dpi values used by X? Or is it that is uses the X values but those are wrong? > It's wrong because > nobody tells the system how big their monitor is these days so it always > picks a DPI that is too small. Here X gets it from the monitor at start-up (via the logic of the video card). I did not realize that my situation is not normal. Somehow, I think setting a proper dpi should ideally be done in X. But in practice ... I agree with you that 95 dpi is about the minimum that is being used today. But I would keep this the minimum: if the dpi is set bigger then it is probably intentional. Either monitor magic or from a careful system administration. Rein. -- Rein Klazes rklazes(a)xs4all.nl

4 4

Re: Add documentation on the address space layout in Wine
by Pouech Eric DMI AEI CAEN May 28, 2004

May 28, 2004

Hi Mike I've started to revisit the architecture.sgml file, and started to write also about this matters (and a few others). For coherency reasons, if you don't mind, I'll reintegrate your bits in the architecture.sgml file (what you wrote covers also process image loading...). A+ > Message du 28/05/04 01:35 > De : "Mike Hearn" > A : wine-patches(a)winehq.com > Copie à : > Objet : Add documentation on the address space layout in Wine > Mike Hearn > Add documentation on the address space layout in Wine > > Generated from: > * mike(a)navi.cx--2004/wine--mainline--0.9--patch-25 > > --- /dev/null 2003-09-15 14:40:47.000000000 +0100 > +++ documentation/address-space.sgml 2004-05-28 00:26:06.000000000 +0100 > @@ -0,0 +1,175 @@ > + > + > + > + > + Every Win32 process in Wine has its own dedicated native process on the host system, and > + therefore its own address space. This section explores the layout of the Windows address space > + and how it is emulated. > + > + > + > + Firstly, a quick recap of how virtual memory works. Physical memory in RAM chips is split > + into frames, and the memory that each process sees is split > + into pages. Each process has its own 4 gigabytes of address space (4gig > + being the maximum space addressable with a 32 bit pointer). Pages can be mapped or unmapped: > + attempts to access an unmapped page cause an EXCEPTION_ACCESS_VIOLATION which has the > + easily recognizable code of 0xC0000005. Any page can be mapped to any frame, therefore you can > + have multiple addresses which actually "contain" the same memory. Pages can also be mapped to > + things like files or swap space, in which case accessing that page will cause a disk access to > + read the contents into a free frame. > + > + > + > + > + > + > + When a Win32 process starts, it does not have a clear address space to use as it pleases. Many pages > + are already mapped by the operating system. In particular, the EXE file itself and any DLLs it > + needs are mapped into memory, and space has been reserved for the stack and a couple of heaps > + (zones used to allocate memory to the app from). Some of these things need to be at a fixed > + address, and others can be placed anywhere. > + > + > + > + The EXE file itself is almost always mapped at address 0x400000 and up: indeed, most EXEs have > + their relocation records stripped which means they must be loaded at their base address and > + cannot be loaded at any other address. > + > + > + > + DLLs are internally much the same as EXE files but they have relocation records, which means > + that they can be mapped at any address in the address space. Remember we are not dealing with > + physical memory here, but rather virtual memory which is different for each > + process. Therefore OLEAUT32.DLL may be loaded at one address in one process, and a totally > + different one in another. Ensuring all the functions loaded into memory can find each other > + is the job of the Windows dynamic linker, which is a part of NTDLL. > + > + > + > + So, we have the EXE and its DLLs mapped into memory. Two other very important regions also > + exist: the stack and the process heap. The process heap is simply the equivalent of the libc > + malloc arena on UNIX: it's a region of memory managed by the OS which malloc/HeapAlloc > + partitions and hands out to the application. Windows applications can create several heaps but > + the process heap always exists. It's created as part of process initialization in > + dlls/ntdll/thread.c:thread_init(). > + > + > + > + There is another heap created as part of process startup, the so-called shared or system > + heap. This is an undocumented service that exists only on Windows 9x: it is implemented in > + Wine so native win9x DLLs can be used. The shared heap is unusual in that anything allocated > + from it will be visible in every other process. This heap is always created at the > + SYSTEM_HEAP_BASE address or 0x65430000 and defaults to a megabyte in size. > + > + > + > + So far we've assumed the entire 4 gigs of address space is available for the application. In > + fact that's not so: only the lower 2 gigs are available, the upper 2 gigs are on Windows NT > + used by the operating system and hold the kernel (from 0x80000000). Why is the kernel mapped > + into every address space? Mostly for performance: while it's possible to give the kernel its > + own address space too - this is what Ingo Molnars 4G/4G VM split patch does for Linux - it > + requires that every system call into the kernel switches address space. As that is a fairly > + expensive operation (requires flushing the translation lookaside buffers etc) and syscalls are > + made frequently it's best avoided by keeping the kernel mapped at a constant position in every > + processes address space. > + > + > + > + On Windows 9x, in fact only the upper gigabyte (0xC0000000 and up) is used by the kernel, the > + region from 2 to 3 gigs is a shared area used for loading system DLLs and for file > + mappings. The bottom 2 gigs on both NT and 9x are available for the programs memory allocation > + and stack. > + > + > + > + There are a few other magic locations. The bottom 64k of memory is deliberately left unmapped > + to catch null pointer dereferences. The region from 64k to 4mb are reserved for DOS > + compatibility and contain various DOS data structures. Finally, the address space also > + contains mappings for the Wine binary itself, any native libaries Wine is using, the glibc > + malloc arena and so on. > + > + > + > + > + > + > + > + > + Up until about the start of 2004, the Linux address space very much resembled the Windows 9x > + layout: the kernel sat in the top gigabyte, the bottom pages were unmapped to catch null > + pointer dereferences, and the rest was free. The kernels mmap algorithm was predictable: it > + would start by mapping files at low addresses and work up from there. > + > + > + > + The development of a series of new low level patches violated many of these assumptions, and > + resulted in Wine needing to force the Win32 address space layout upon the system. This > + section looks at why and how this is done. > + > + > + > + The exec-shield patch increases security by randomizing the kernels mmap algorithms. Rather > + than consistently choosing the same addresses given the same sequence of requests, the kernel > + will now choose randomized addresses. Because the Linux dynamic linker (ld-linux.so.2) loads > + DSOs into memory by using mmap, this means that DSOs are no longer loaded at predictable > + addresses, so making it harder to attack software by using buffer overflows. It also attempts > + to relocate certain binaries into a special low area of memory known as the ASCII armor so > + making it harder to jump into them when using string based attacks. > + > + > + > + Prelink is a technology that enhances startup times by precalculating ELF global offset > + tables then saving the results inside the native binaries themselves. By grid fitting each > + DSO into the address space, the dynamic linker does not have to perform as many relocations > + so allowing applications that heavily rely on dynamic linkage to be loaded into memory much > + quicker. Complex C++ applications such as Mozilla, OpenOffice and KDE can especially benefit > + from this technique. > + > + > + > + The 4G VM split patch was developed by Ingo Molnar. It gives the Linux kernel its own address > + space, thereby allowing processes to access the maximum addressable amount of memory on a > + 32-bit machine: 4 gigabytes. It allows people with lots of RAM to fully utilise that in any > + given process at the cost of performance: as mentioned previously the reason behind giving > + the kernel a part of each processes address space was to avoid the overhead of switching on > + each syscall. > + > + > + > + Each of these changes alter the address space in a way incompatible with Windows. Prelink and > + exec-shield mean that the libraries Wine uses can be placed at any point in the address > + space: typically this meant that a library was sitting in the region that the EXE you wanted > + to run had to be loaded (remember that unlike DLLs, EXE files cannot be moved around in > + memory). The 4G VM split means that programs could receive pointers to the top gigabyte of > + address space which some are not prepared for (they may store extra information in the high > + bits of a pointer, for instance). In particular, in combination with exec-shield this one is > + especially deadly as it's possible the process heap could be allocated beyond > + ADDRESS_SPACE_LIMIT which causes Wine initialization to fail. > + > + > + > + The solution to these problems is for Wine to reserve particular parts of the address space > + so that areas that we don't want the system to use will be avoided. We later on > + (re/de)allocate those areas as needed. One problem is that some of these mappings are put in > + place automatically by the dynamic linker: for instance any libraries that Wine > + is linked to (like libc, libwine, libpthread etc) will be mapped into memory before Wine even > + gets control. In order to solve that, Wine overrides the default ELF initialization sequence > + at a low level and reserves the needed areas by using direct syscalls into the kernel (ie > + without linking against any other code to do it) before restarting the standard > + initialization and letting the dynamic linker continue. This is referred to as the > + preloader and is found in ld-winepreload.so (loader/preloader.c) > + > + > + > + Once the usual ELF boot sequence has been completed, some native libraries may well have been > + mapped above the 3gig limit: however, this doesn't matter as 3G is a Windows limit, not a > + Linux limit. We still have to prevent the system from allocating anything else above there > + (like the heap or other DLLs) though so Wine performs a binary search over the upper gig of > + address space in order to iteratively fill in the holes with MAP_NORESERVE mappings so the > + address space is allocated but the memory to actually back it is not. This code can be found > + in libs/wine/mmap.c:reserve_area. > + > + > + > + > + > --- documentation/wine-devel.sgml > +++ documentation/wine-devel.sgml > @@ -13,6 +13,7 @@ > > > > + > > > > @@ -138,6 +139,7 @@ > &implementation; > &porting; > &consoles; > + &address-space; > &cvs-regression; > > > > >

1 0

[Fwd: winetest: copy and submit manually]
by Robert Reif May 28, 2004

May 28, 2004

oops, wrong list.

1 0

RE: FW: Edit Paste operation does not behave same as MS Windows
by Krishna Murthy May 28, 2004

May 28, 2004

This means it is sufficient to return FALSE in EDIT_MakeFit if the honor limit is not set and size exceeds the buffer limit? static BOOL EDIT_MakeFit(EDITSTATE *es, UINT size, BOOL honor_limit) { HLOCAL hNew32W; if ((!honor_limit) && (es->buffer_limit > 0) && (size > es->buffer_limit)) { EDIT_NOTIFY_PARENT(es, EN_MAXTEXT, "EN_MAXTEXT"); return FALSE; } ... } I will verify with this change and if works in all condition, will submit the patch. Thanks for your help, Krishna -----Original Message----- From: Alexandre Julliard [mailto:julliard(a)winehq.org] Sent: Thursday, May 27, 2004 5:12 PM To: Krishna Murthy Cc: wine-devel(a)winehq.org Subject: Re: FW: Edit Paste operation does not behave same as MS Windows Krishna Murthy <Krishna.Murthy(a)guptaworldwide.com> writes: > Actually the logic truncates the data which exceeded the buffer. If > the honor limit is set and the size exceeded, the EDIT_MakeFit will return TRUE. > The following code after the EDIT_MakeFit will try to take limit to > the buffer size. > > + /* Only replace or insert that many characters that can fit into the > + edit control. Do this only if honor_limit is TRUE */ > + if(honor_limit && strl > es->buffer_limit) > + strl = es->buffer_limit - (tl - (e-s)); That only takes into account the limit, not the current buffer size. It's very possible that you need to grow the buffer even when the limit is not exceeded. Also you only check the length of the inserted string against the limit, you should check the total resulting length instead. -- Alexandre Julliard julliard(a)winehq.org

1 0

RE: FW: Edit Paste operation does not behave same as MS Windows
by Krishna Murthy May 28, 2004

May 28, 2004

Actually the logic truncates the data which exceeded the buffer. If the honor limit is set and the size exceeded, the EDIT_MakeFit will return TRUE. The following code after the EDIT_MakeFit will try to take limit to the buffer size. + /* Only replace or insert that many characters that can fit into the + edit control. Do this only if honor_limit is TRUE */ + if(honor_limit && strl > es->buffer_limit) + strl = es->buffer_limit - (tl - (e-s)); Krishna -----Original Message----- From: Alexandre Julliard [mailto:julliard(a)winehq.org] Sent: Thursday, May 27, 2004 2:30 PM To: Krishna Murthy Cc: wine-devel(a)winehq.org Subject: Re: FW: Edit Paste operation does not behave same as MS Windows Krishna Murthy <Krishna.Murthy(a)guptaworldwide.com> writes: > - Change log Entry: > EDIT_MakeFit should return TRUE when the honor_limit is set and paste > buffer exceeds the limit. Also EDIT_EM_ReplaceSel should compute size > which could fit in the edit control by truncating the extra characters > to behave similar to MS Windows paste operation. You are bypassing the reallocation of the buffer when the limit is exceeded, but still adding the data to the buffer. I don't think this will work right. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: Crash and corrupted registry
by Alexandre Julliard May 27, 2004

May 27, 2004

Christian Costa <titan.costa(a)wanadoo.fr> writes: > This seems to be more a filesytem issue then because the file is in a > really bad shape. > I use reiserfs on a Mandrake 2.4.21-0.13mdk kernel. > It seems to happend on files that are open when the crash (freeze + > reboot) occurs. Yes, it very much looks like a filesystem issue, I prefer that <g> If you feel like trying to reproduce it, maybe you could try to add an fsync() in the saving code and see if this makes a difference. -- Alexandre Julliard julliard(a)winehq.org

3 3

Re: FW: Edit Paste operation does not behave same as MS Windows
by Alexandre Julliard May 27, 2004

May 27, 2004

Krishna Murthy <Krishna.Murthy(a)guptaworldwide.com> writes: > - Change log Entry: > EDIT_MakeFit should return TRUE when the honor_limit is set and paste buffer > exceeds the limit. Also EDIT_EM_ReplaceSel should compute size which could > fit in the edit control by truncating the extra characters to behave similar > to MS Windows paste operation. You are bypassing the reallocation of the buffer when the limit is exceeded, but still adding the data to the buffer. I don't think this will work right. -- Alexandre Julliard julliard(a)winehq.org

1 0