Currently there are quite a few places in ROS src where 64-bit integral
constants are either hard-coded to be GNU C only (?), or almost worse
littered with conditional compilation.
Neither is satisfactory. 1 is making it impossible to compile using e.g. an
MS compiler, and 2 makes the code so ugly that... Well, you understand.
So, in the interest of fixing this once and for all, the only option is to
name a macro to handle these compiler specifics in one place, and then use
that macro when defining 64-bit integral constants. The question is then,
what to name it? The macro will be defined something like (omitting unsigned
here):
#if defined __GNUC__
#define ZZZ(X) X##LL
#elif defined _MSC_VER
#define ZZZ(X) X##i64
#else
#error Unknown compiler for 64-bit integral constant suffix
#endif
Serious suggestions that have surfaced so far are:
DEFINE_INT64
MAKE_INT64
If suggesting something else, keep in mind that
- while short might be beautiful (and fast to type), using short macros is
likely to sooner or later create conflicts.
- leading underscore followed by capital character or another underscore is
not legal for C++.
- It's to make it clear that it's a 64-bit int defined, why suggestions
mentioning LL or i64, as in compiler-specific extensions/implementation
details, are likely not to help or be accepted since that's exactly what we
want to get away from.
Please comment.
/Mike
gcc -I./include -I../include -D__NTOSKRNL__ -Wall -Werror
-D_DISABLE_TIDENTS -
D_SEH_NO_NATIVE_NLG -D__3GB__ -I../include -I../w32api/include -pipe
-march=i486
-D_M_IX86 -D__NTOSKRNL__ -I. -D_SEH_NO_NATIVE_NLG -I../include
-I../w32api/incl
ude -pipe -march=i486 -D_M_IX86 -D__NTOSKRNL__ -I. -D_SEH_NO_NATIVE_NLG
-I../inc
lude -I../w32api/include -pipe -march=i486 -D_M_IX86 -c ke/i386/exp.c -o
ke/i386
/exp.o
ke/i386/exp.c: In function `KiTrapHandler':
ke/i386/exp.c:579: error: `ATTEMPTED_EXECUTE_OF_NOEXECUTE_MEMORY'
undeclared (fi
rst use in this function)
ke/i386/exp.c:579: error: (Each undeclared identifier is reported only once
ke/i386/exp.c:579: error: for each function it appears in.)
mingw32-make[1]: *** [ke/i386/exp.o] Error 1
mingw32-make[1]: Leaving directory `C:/reactos/ntoskrnl'
mingw32-make: *** [ntoskrnl] Error 2
C:\reactos>gcc --version
gcc (GCC) 3.4.1 (mingw special)
Copyright (C) 2004 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
I've tried my best to make a working implementaiton of
StackAttach/StackDetach, because it's almost one of the most important
APIs in NT5+ Drivers. I think I've got Detach working properly, but
Attach will require some changes to KeAttach itself so that it can save
the old ApcState & other things. I might get around to fixing it, but at
least the API should work now and allow more drivers to run sucesfully.
Also, if any Intel experts woudl like to check out KeFlushCurrentTb... I
followed the Intel Manual specs, just want to make sure I did it
properly...my GNU Assembler syntax sucks.
Best regards,
Alex Ionescu
ion(a)cvs.reactos.com wrote:
>CVSROOT: /CVS/ReactOS
>Module name: reactos
>Repository: reactos/ntoskrnl/ke/
>Changes by: ion(a)mok.osexperts.com 04/10/12 18:42:15
>
>Modified files:
> reactos/include/ddk/: kefuncs.h
> reactos/ntoskrnl/: Makefile.i386
> reactos/ntoskrnl/include/internal/: ke.h ps.h
> reactos/ntoskrnl/ke/: apc.c device.c dpc.c kthread.c process.c
>
>Log message:
> Implemented KeAreApcsDisabled, KeFlushEntireTb, KeIsExecutingDpc, KeSetKernelStackSwapEnable, KeQueryPriorityThread, KeRevertToUserAffinityThread, KeSetIdealProcessorThread, KeSetSystemAffinityThread, KeTerminateThread, KeStackAttachProcess, KeUnstackDetachProcess
>
>_______________________________________________
>Ros-cvs mailing list
>Ros-cvs(a)reactos.com
>http://reactos.com/mailman/listinfo/ros-cvs
>
>
>
Hello,
For almost a week now my symbols have been missing. It started
with gdb not reading in any symbols when I was trying to hunt down
cygwin problems, then I re-enabled KDBG in config and rebuilt and KDBG
can tell you the function name, but not the file name and line number.
Last night, I deleted my sandbox and checked out a fresh copy from
CVS, only enabling DBG. I think it was Arty who suggested I check to
see if the targets are getting the "-g" flag. I captured the entire
build into a log, and these are the only targets that had "-g" on the
gcc command line:
ntdll (-g)
opengl32 (-g3)
dnsapi (-g)
bzip2 (-g)
oskittcp (-g)
patblt (-g)
primitives (-g)
strechblt (-g)
txtscale (-g)
wm_erasebkgnd (-g)
I checked this against my symbols directory and they correspond to
.sym files that appear to be valid. All the others are 16 bytes, and
don't appear to contain any useful information.
I'm not much of a make guru, so I thought I would report it rather
than continue to hunt it down alone. ;0)
Thanks,
Andrew
--
The cheese stands alone.
Hi!
Since I have seen mesa32 crash with SSE enabled I looked into fixing it
a bit...
When I was looking at ntoskrnl/ke/i386/tskswitch.S I found FIXMEs for
debug/FPU save/restore.
Now I am wondering where should this information be stored? I think it
should be part of the TSS (include/ntos/tss.h) but I am not sure.
For SSE support the SSE registers would also have to be saved/restored
on task switches and some other stuff which I do not yet know I think.
Maybe somebody can tell me where the FPU state should be stored so I can
try to implement it.
Thanks,
blight
Hi,
Anyone against switching ROS to be compiled by default on the 2GB
boundary? (This can always be locally changed by /3GB = 1)
For the sake of consistency and perhaps better driver support, as has
been found in the past.
Then we can start working on a real freeldr PE loader...
Best regards,
Alex Ionescu
Why is CVSspam output generated for only some of the ReactOS developers ?
Reading what was changed is very interesting for me
and CVSspam produces very nice diff.
Thanks, David
--
David Kredba <kredba at ibot.cas.cz>
GPG: ID 1024D/5B6B02DE
Fingerprint: F0B3312596BEDCF91DFB 0699E06AACD75B6B02DE
Hi James,
please update to the latest version, this has been fixed.
see
http://cvs.reactos.com/cgi-bin/cvsweb.cgi/reactos/subsys/system/winefile/it.
rc.diff?r1=1.2&r2=1.3
-----Message d'origine-----
De : James Tabor
[mailto:jimtabor@adsl-64-217-116-74.dsl.hstntx.swbell.net]
Envoyé : samedi 9 octobre 2004 03:17
À : ros-dev(a)reactos.com
Objet : [ros-dev] Re: [ros-cvs] CVS Update: reactos
mf(a)cvs.reactos.com wrote:
> CVSROOT: /CVS/ReactOS
> Module name: reactos
> Repository: reactos/subsys/system/winefile/
> Changes by: mf(a)mok.osexperts.com 04/10/08 13:20:15
>
> Modified files:
> reactos/subsys/system/winefile/: winefile.c winefile.h
>
> Log message:
> call vswprintf@msvcrt in Wine's UNICODE version
>
i386-mingw32-windres -D__USE_W32API -DNDEBUG -DWIN32 -D_ROS_ -D__WINDRES__
-DUNICODE --include-dir
../../../include --include-dir ../../../w32api/include winefile.rc -o
winefile.coff
i386-mingw32-windres: it.rc:184: parse error
make: *** [winefile.coff] Error 1
_______________________________________________
Ros-dev mailing list
Ros-dev(a)reactos.com
http://reactos.com:8080/mailman/listinfo/ros-dev
----------------------------------------------------------------------------
Ce message ainsi que toutes pièces jointes (le "message") sont confidentiels
et sont exclusivement destinés à l'usage de la personne à laquelle ils sont
adressés. Tout point de vue ou toute opinion contenus dans ce message
expriment la pensée personnelle de leur auteur et ne représentent pas
nécessairement la position des sociétés du Groupe GEFCO. Si vous n'êtes pas
la personne à laquelle ce message est destiné, veuillez noter que vous avez
reçu cet e-mail par erreur et qu'il vous est strictement interdit
d'utiliser, de diffuser, de transférer, d'imprimer ou de copier ce message.
Si vous avez reçu ce message par erreur, merci de contacter la personne qui
vous l'a adressé et de l'effacer immédiatement. Les sociétés du Groupe GEFCO
déclinent toute responsabilité en cas d'altération, de modification,
d'édition, de diffusion sans autorisation de ce message ou en cas
d'affection de ce message par un virus.
This message and any attachments (the "message") are confidential and
intended solely for the use of the individual to whom they are addressed.
Any views or opinions presented are solely those of the author and do not
necessarily represent those of the GEFCO Group of Companies. If you are not
the intended recipient, be advised that you have received this email in
error and that any use, dissemination, forwarding, printing, or copying of
this message is strictly prohibited. If you have received this message in
error please contact the sender and delete the message immediately. The
GEFCO Group of Companies shall not be liable for the message if altered,
changed, falsified, edited, diffused without authorization or affected by
any virus.
----------------------------------------------------------------------------
Richard and I can't get the ros cvs build to boot...I get stuck on a
black screen, and he gets a freeze on ndis.sys as well as a blackscreen
if running the bootcd.
Can anyone confirm?
Best regards,
Alex Ionescu
Hi arty,
Immediately after your answer on #reactos,
I did make clean in drivers/lib/ip and drivers/net/tcpip.
The problem is still here.
I even tried doing make clean in drivers/ to be sure.
Usurp.
"You believe its the year 2004,
when in fact it's closer to 1984."
Matrix / 1984.
-----Message d'origine-----
De : art yerkes [mailto:ayerkes@speakeasy.net]
Envoyé : vendredi 8 octobre 2004 03:50
À : ReactOS Development List
Cc : jimtabor(a)adsl-64-217-116-74.dsl.hstntx.swbell.net
Objet : Re: [ros-dev] Re: [ros-cvs] CVS Update: reactos tcpip
On Fri, 08 Oct 2004 00:54:07 +0000
James Tabor <jimtabor(a)adsl-64-217-116-74.dsl.hstntx.swbell.net> wrote:
You need to make clean in drivers/lib/ip and drivers/net/tcpip. Hopefully
this will be the last time.
--
Hey, Adam Smith, keep your invisible hands to yourself!
_______________________________________________
Ros-dev mailing list
Ros-dev(a)reactos.com
http://reactos.com:8080/mailman/listinfo/ros-dev
----------------------------------------------------------------------------
Ce message ainsi que toutes pièces jointes (le "message") sont confidentiels
et sont exclusivement destinés à l'usage de la personne à laquelle ils sont
adressés. Tout point de vue ou toute opinion contenus dans ce message
expriment la pensée personnelle de leur auteur et ne représentent pas
nécessairement la position des sociétés du Groupe GEFCO. Si vous n'êtes pas
la personne à laquelle ce message est destiné, veuillez noter que vous avez
reçu cet e-mail par erreur et qu'il vous est strictement interdit
d'utiliser, de diffuser, de transférer, d'imprimer ou de copier ce message.
Si vous avez reçu ce message par erreur, merci de contacter la personne qui
vous l'a adressé et de l'effacer immédiatement. Les sociétés du Groupe GEFCO
déclinent toute responsabilité en cas d'altération, de modification,
d'édition, de diffusion sans autorisation de ce message ou en cas
d'affection de ce message par un virus.
This message and any attachments (the "message") are confidential and
intended solely for the use of the individual to whom they are addressed.
Any views or opinions presented are solely those of the author and do not
necessarily represent those of the GEFCO Group of Companies. If you are not
the intended recipient, be advised that you have received this email in
error and that any use, dissemination, forwarding, printing, or copying of
this message is strictly prohibited. If you have received this message in
error please contact the sender and delete the message immediately. The
GEFCO Group of Companies shall not be liable for the message if altered,
changed, falsified, edited, diffused without authorization or affected by
any virus.
----------------------------------------------------------------------------
Hello,
OpenOffice will load if you copy a working install from a Windows box
but we have some major problems in painting in the program. Is anyone
interested in looking at it?
Thanks
Steven
__________________________________
Do you Yahoo!?
Yahoo! Mail Address AutoComplete - You start. We finish.
http://promotions.yahoo.com/new_mail
Hello,
Arty and I played around a bit with the busybox dhcpd for Windows and I
was able to get it to run under Wine and give a address to my IP phone
and Art said that it should work under ReactOS once we implement
gethostbyname and gethostname. Does anyone want to take a stab at
implementing these? Art has said that a good deal of the functionality
is implemented in dnsapi.
Thanks
Steven
_______________________________
Do you Yahoo!?
Declare Yourself - Register online to vote today!
http://vote.yahoo.com
> From: Ge van Geldorp
>
> Log message:
> -Don't define COBJMACROS in objbase.h.
> Update the Wine sources accordingly.
I know this will cause warnings during shell32 compilation ("warning:
"COBJMACROS" redefined"), will fix this with the next Wine sync.
Gé van Geldorp.
<ion(a)cvs.reactos.com> wrote:
> CVSROOT: /CVS/ReactOS
> Module name: reactos
> Repository: reactos/ntoskrnl/mm/
> Changes by: ion(a)mok.osexperts.com 04/10/04 14:38:49
>
> Modified files:
> ./: config rules.mak
> reactos/include/ntdll/: rtl.h
> reactos/ntoskrnl/: Makefile
> reactos/ntoskrnl/include/internal/i386/: mm.h
> reactos/ntoskrnl/ke/i386/: multiboot.S
> reactos/ntoskrnl/mm/: mminit.c
>
> Log message:
> Un-hardcoded the 0xC0000000 value, and stuck the define in one single
> place both for ASM and C code. This define is 0x80000000 by default, but
> automatically sets itself to 0xC0000000 if 3GB is set to 1 in
> reactos/config. This option is turned on by default for now, because the
> Default heap is at 0xA0000000, which doesn't work on a non 3GB system.
> Lower values seem to crash Win32K but I fill figure out a solution.
I tried a NULL base address in the call to RtlCreateHeap (in
lib/ntdll/ldr/startup.c) and regedit (WIN32 app) works very well. The NULL
base address enables the default bottom-up memory allocation so the heap
will be allocated from the bootom of the process address space just like
Windows does. HEAP_BASE should have been removed ages ago.
Regards,
Eric
Is Sanos`s implementation of ndis and rtl8139 driver etc.
interesting for ReactOS developers ?
http://www.jbox.dk/sanos/index.htm
Regards, David
--
David Kredba <kredba at ibot.cas.cz>
GPG: ID 1024D/5B6B02DE
Fingerprint: F0B3312596BEDCF91DFB 0699E06AACD75B6B02DE
>From current CVS, running in whole-page alloc mode, I see a buffer
overrun in lib/rtl/acl.c early in the boot process. What I see is
a few calls to RtlpAddKnownAce which slowly overgrow the available
buffer. The following patch fixes it, but I don't know if it's
right as I'm not sure if AclSize should be adjusted. The reason
a patch like this is needed is that AclSize is checked against
the size of the Sid to be copied in, but that size never changes
so as many sids as desired could be copied in without the check
at line 136 going off, thus overrunning available space.
--
Hey, Adam Smith, keep your invisible hands to yourself!
Hi Hartmut,
Thanks for the commit, I wasn't sure how to fix those!
Best Regards,
Alex Ionescu
Hartmut Birr wrote:
>CVSROOT: /CVS/ReactOS
>Module name: reactos
>Repository: reactos/ntoskrnl/ke/i386/
>Changes by: hbirr(a)mok.osexperts.com 04/10/05 11:53:28
>
>Modified files:
> reactos/ntoskrnl/mm/i386/: page.c
> reactos/ntoskrnl/ke/i386/: multiboot.S
>
>Log message:
> - Removed some hard coded values, which depend on a kernel base of 0xc0000000.
>
>_______________________________________________
>Ros-cvs mailing list
>Ros-cvs(a)reactos.com
>http://reactos.com/mailman/listinfo/ros-cvs
>
>
>
Hi,
I just want to make a few annotations to the new dhcpd.
Why do we have to add these useless files, like shell scripts, makefile.am,
makefile.in, etc. to the CVS tree if we want to make dhcpd a real service?
The source code will loose its portability anyway. And why keep this rather
complex directory structure for such a simple utility?
Btw, the dhcp server looks like a hack to me. It is leaking lots of memory
because there is not a single call to free() in the source and the dhcp
options handling it pretty crude. And it doesn't seem to work with a Win2K
client.
I'd suggest to use the Moreton dhcp server (dhcp-win-0.8.25.tar) instead. It
has got a much better internal structure and it works properly. I ran it on
Win98SE and tested it with a Win2K client. The only disadvantage of the
Moreton server is that it uses a binary config file.
Regards,
Eric
Hi,
why is the drive letter added to users path if the install drive isn't C?
CreateDirectory("G:\Documents and
Settings\Administrator.REACTOS_G\Application Data\Microsoft")
CreateDirectory("G:\Documents and
Settings\Administrator.REACTOS_G\Application Data\Microsoft\Internet
Explorer")
CreateDirectory("G:\Documents and
Settings\Administrator.REACTOS_G\Application Data\Microsoft\Internet
Explorer\Quick Launch")
- Hartmut
