Ros-dev

ros-dev@reactos.org

5790 discussions

Start a nNew thread

Need someone to upload patch for me
by Neil Blackburn 05 Jan '05

05 Jan '05

Fixed a few spelling mistakes The patch is in attachment. Thank You!

2 1

[PATCH] Fix LUID constants
by Nedko Arnaudov 05 Jan '05

05 Jan '05

The attached patch fixes "warnings + -Werror = not compile" for rosapps/tests/tokentest/tokentest.c tokentest.c: In function `CreateInitialSystemToken': tokentest.c:359: warning: braces around scalar initializer tokentest.c:359: warning: (near initialization for `authId.LowPart') tokentest.c:359: warning: excess elements in scalar initializer tokentest.c:359: warning: (near initialization for `authId.LowPart') I think SYSTEM_LUID and similar macros for initial values are intended as initializers of LUID variables, not LARGE_INTEGER ones. I've posted this (and not commited it myself) because I can be missing something important. Is there other issue with LUID <-> LARGE_INTEGER ? <#secure method=pgpmime mode=sign> -- Nedko Arnaudov <GnuPG KeyID: DE1716B0>

1 0

Re:Re: [ros-dev] Re: [ros-diffs] [ion] 12774: Add more Critical Section
by yang 05 Jan '05

05 Jan '05

Alex Ionescu wrote: > > > > Well, yes, win32. I've been setup before, but I haven't > > kept up with the tool chain. > > > > Do we still need the djcc stuff for building freeloader? > > > > What would be really cool was an MSI that would get me > > a complete 'ReactoOS' build environment in one simple > > step. Maybe I'll look into it. > > I'm currently working on this exact project! I'm a newbie. I'm looking for it. Can you tell me something about it?? Thanks. yang ___________________ Come from China

1 0

Re: [ros-diffs] [ion] 12774: Add more Critical Section Debug Data and fix two bugs.
by Joseph Galbraith 05 Jan '05

05 Jan '05

ion(a)svn.reactos.com wrote: > + DPRINT("Waiting on Critical Section: %x\n", CriticalSection); > + if (CriticalSection->DebugInfo) CriticalSection->DebugInfo->EntryCount++; I'm just reading the diffs here, so I could be way off, but shouldn't this use an InterlockedIncrement... otherwise you could get a context switch between in the middle of your increment to another threading attempting to lock the critical section. It is just debug information, but still... Also, looking at the rest of the code, I notice that spin count doesn't appear to be implemented. I don't have a build environment even setup, so take the attached patch with a grain of salt. The changes in RtlpCreateCriticalSEctionSem() are because the InterlockCompareExchangePointer() should already have written the new event into the data structure. Maybe on day I'll manage to get my build environment setup again. - Joseph Index: C:/Users/galb/ReactOS/reactos/lib/ntdll/rtl/critical.c =================================================================== --- C:/Users/galb/ReactOS/reactos/lib/ntdll/rtl/critical.c (revision 12774) +++ C:/Users/galb/ReactOS/reactos/lib/ntdll/rtl/critical.c (working copy) @@ -133,7 +133,17 @@ { HANDLE Thread = (HANDLE)NtCurrentTeb()->Cid.UniqueThread; - /* Try to Lock it */ + /* Try for SpinCount times to Lock it */ + for(LONG i = 0; i < CriticalSection->SpinCount; i++) + { + if ( RtlTryEnterCriticalSection(CriticalSection) ) + return STATUS_SUCCESS; + } + + /* + * Still didn't get it; we have to bump the LockCount, which + * might get it for us, and then wait + */ if (InterlockedIncrement(&CriticalSection->LockCount)) { /* @@ -398,12 +408,12 @@ /* Increase the Debug Entry count */ DPRINT("Waiting on Critical Section: %x\n", CriticalSection); - if (CriticalSection->DebugInfo) CriticalSection->DebugInfo->EntryCount++; + if (CriticalSection->DebugInfo) InterlockedIncrement(&CriticalSection->DebugInfo->EntryCount); for (;;) { /* Increase the number of times we've had contention */ - if (CriticalSection->DebugInfo) CriticalSection->DebugInfo->ContentionCount++; + if (CriticalSection->DebugInfo) InterlockedIncrement(&CriticalSection->DebugInfo->ContentionCount); /* Wait on the Event */ DPRINT("Waiting on Event: %x\n", CriticalSection->LockSemaphore); @@ -523,21 +533,14 @@ return; } - if (!(hEvent = InterlockedCompareExchangePointer((PVOID*)&CriticalSection->LockSemaphore, - (PVOID)hNewEvent, - 0))) { - - /* We created a new event succesffuly */ - hEvent = hNewEvent; - } else { - - /* Some just created an event */ - DPRINT("Closing already created event!\n"); - NtClose(hNewEvent); + + if ( InterlockedCompareExchangePointer((PVOID*)&CriticalSection->LockSemaphore, + (PVOID)hNewEvent, + 0)) != 0) { + /* Someone else just created the event */ + NtClose(hNewEvent); } - - /* Set either the new or the old */ - CriticalSection->LockSemaphore = hEvent; + DPRINT("Event set!\n"); }

4 7

PSID -> PISID and /rtl /ntoskrnl/se duplication.
by Alex Ionescu 04 Jan '05

04 Jan '05

Hi, Attached is a patch with uses the proper PSID/PISID types. Also, I've noticed that we currently have duplicated code between /lib/rtl and /ntoskrnl/se. The removals are also in this patch. Is it OK to remove them, or is there some reason they must be duplicated? Best regards, Alex Ionescu

1 0

I need help to build reactos to develop reactos.
by yang 04 Jan '05

04 Jan '05

Hi I am a new comer. I began to read codes a few days ago, now I installed reactos 0.2.4 in vmware, but it can't do any thing even if I want to copy a file(both command line and GUI), when I do this in GUI, it show me a blue face or do nothing. what do you make up your reactos develop system? It means read, edit, and compile code in reactos. I was on win 32, which emulater is better to run reactos on it? any idea would be appreciated.

2 1

object types (was [ros-svn] [ion] 12748: Fix incorrect LPC Object ...)
by Hartmut Birr 04 Jan '05

04 Jan '05

Hi, I've add the following line to my test driver: extern __attribute__((dllimport)) POBJECT_TYPE* ExEventObjectType; DbgPrint("%x %x %x %x\n", ExEventObjectType, *(PULONG)ExEventObjectType, MmGetSystemRoutineAddress(&NameU), *(PULONG)MmGetSystemRoutineAddress(&NameU)); Running on W2K, I get this result: GCC: 81ebd540 81ebd440 8047a888 81ebd540 MSVC: 8047a888 81ebd540 8047a888 81ebd540 Running on ROS: GCC: c0692ef8 54545645 c00e0760 c0692ef8 MSVC: c00e0760 c0692ef8 c00e0760 c0692ef8 It seems that GCC and MSVC does handle imported data symbols different. In MSVC is an imported data symbol a pointer to the variable and in GCC the value of the variable. - Hartmut

5 4

Re: 12775: don't pass vector<pointer> by value unless you are looking for trouble, also fixed Assert() to take stdargs to make it's usage simpler and less error-prone. Also fixed IsTrue(), etc, to take file and line info by args and created macros to pass that info automatically.
by Royce Mitchell III 04 Jan '05

04 Jan '05

royce(a)svn.reactos.com wrote: >don't pass vector<pointer> by value unless you are looking for trouble, also fixed Assert() to take stdargs to make it's usage simpler and less error-prone. Also fixed IsTrue(), etc, to take file and line info by args and created macros to pass that info automatically. > > > Casper, I made some changes to Assert() and the code that calls it, in the hope that this is what you really wanted. I apologize sincerely if you had wanted to do something else with message[], but if you do, I think it would be easier to do so from within Assert(). I will revert that part of this patch if you needed the buffer in Assert()'s callers.

1 0

Re:[ros-svn] [ion] 12748: Fix incorrect LPC Object export...we export pointers not the object itself! Also remove more TIME stuf
by ea＠iol.it 04 Jan '05

04 Jan '05

> Fix incorrect LPC Object export...we export pointers not the object itself! Why? Aren't DATA symbols exported by DLLs pointers? Emanuele ____________________________________________________________ Libero ADSL: 3 mesi gratis e navighi a 1.2 Mega. E poi hai l'Adsl senza limiti a meno di 1 euro al giorno. Abbonati subito senza costi di attivazione su http://www.libero.it

6 11

Re: 12739: Add sample build script.
by Martin Fuchs 03 Jan '05

03 Jan '05

Hi, if you want to replace the Makefile build system, please consider using bjam: http://boost.org/tools/build/jam_src/ It has the following advantages: - automatic dependency checking - resolves dependencies in memory without invoking multiple levels of Makefiles - supports nearly every compiler toolset out there, see: http://boost.org/more/getting_started.html#Tools On 02.01.2005 23:17:03 chorns(a)svn.reactos.com wrote: > Add sample build script. > Added files: > branches/xmlbuildsystem/reactos/ReactOS.xml

3 3

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-dev