patch for ia64
This patch makes the radeon driver treat ia64 as a non-legacy architecture. This patch by Alex Williamson. "xc/programs/Xserver/hw/xfree86/drivers/ati/Imakefile assumes ia64 should behave the same as i386 and allows random probing or I/O port space. In moving away from the legacy model, I believe we should change this to set ATIAvoidCPIO to YES. Systems that do no provide a legacy I/O port range may otherwise machine check." --- xc-old/programs/Xserver/hw/xfree86/drivers/ati/Imakefile2003-07-25 06:04:31.0 -0500 +++ xc/programs/Xserver/hw/xfree86/drivers/ati/Imakefile2003-08-07 04:35:24.0 -0500 @@ -82,7 +82,6 @@ * ATIAvoidNonPCI necessarily implies ATIAvoidCPIO. */ #if defined(i386Architecture) || \ -defined(ia64Architecture) || \ defined(AMD64Architecture) || \ defined(AlphaArchitecture) # ifndef ATIAvoidCPIO -- Treasurer, GOLUM, Inc. http://www.golum.org ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
patch for ia64
This patch makes the radeon driver treat ia64 as a non-legacy architecture. This patch by Alex Williamson. "xc/programs/Xserver/hw/xfree86/drivers/ati/Imakefile assumes ia64 should behave the same as i386 and allows random probing or I/O port space. In moving away from the legacy model, I believe we should change this to set ATIAvoidCPIO to YES. Systems that do no provide a legacy I/O port range may otherwise machine check." --- xc-old/programs/Xserver/hw/xfree86/drivers/ati/Imakefile2003-07-25 06:04:31.0 -0500 +++ xc/programs/Xserver/hw/xfree86/drivers/ati/Imakefile2003-08-07 04:35:24.0 -0500 @@ -82,7 +82,6 @@ * ATIAvoidNonPCI necessarily implies ATIAvoidCPIO. */ #if defined(i386Architecture) || \ -defined(ia64Architecture) || \ defined(AMD64Architecture) || \ defined(AlphaArchitecture) # ifndef ATIAvoidCPIO -- Treasurer, GOLUM, Inc. http://www.golum.org ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
Re: patch for ia64
On Sun, 10 Aug 2003, Warren Turkal wrote: > This patch makes the radeon driver treat ia64 as a non-legacy architecture. > This patch by Alex Williamson. > "xc/programs/Xserver/hw/xfree86/drivers/ati/Imakefile assumes ia64 > should behave the same as i386 and allows random probing or I/O > port space. In moving away from the legacy model, I believe we > should change this to set ATIAvoidCPIO to YES. Systems that do > no provide a legacy I/O port range may otherwise machine check." > --- xc-old/programs/Xserver/hw/xfree86/drivers/ati/Imakefile2003-07-25 > 06:04:31.0 -0500 > +++ xc/programs/Xserver/hw/xfree86/drivers/ati/Imakefile2003-08-07 > 04:35:24.0 -0500 > @@ -82,7 +82,6 @@ > * ATIAvoidNonPCI necessarily implies ATIAvoidCPIO. > */ > #if defined(i386Architecture) || \ > -defined(ia64Architecture) || \ > defined(AMD64Architecture) || \ > defined(AlphaArchitecture) > # ifndef ATIAvoidCPIO Also rejected. Alex's assertion no longer holds now that all the major Itanium chipsets are properly supported. Besides, ATIAvoidCPIO (and ATIAvoidNonPCI) are merely stopgaps which I intend to deprecate. You seem to be "submitting" a string of old patches, some of them twice. Why? The proper patch submission mechanism is patch@, or bugs.xfree86.org. Marc. +--+---+ | Marc Aurele La France | work: 1-780-492-9310 | | Computing and Network Services | fax:1-780-492-1729 | | 352 General Services Building | email: [EMAIL PROTECTED] | | University of Alberta +---+ | Edmonton, Alberta | | | T6G 2H1 | Standard disclaimers apply| | CANADA | | +--+---+ XFree86 Core Team member. ATI driver and X server internals. ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
Re: patch for ia64
On Sun, Aug 10, 2003 at 09:40:43PM -0600, Marc Aurele La France wrote: > You seem to be "submitting" a string of old patches, some of them twice. > Why? These patches are all from the Debian packages. -- Daniel Stone <[EMAIL PROTECTED]> http://www.debian.org - http://www.kde.org - http://www.xwin.org "Jeff doesn't use pants often." -- Pia Smith pgp0.pgp Description: PGP signature
patch for ia64 page size
Here is a patch for page size problems on ia64 and radeon. Patch by Chris Ahna: Fixes critical page size problems on ia64 architecture. See following URL for details: https://external-lists.valinux.com/archives/linux-ia64/2001-August/002037.html Comment by [EMAIL PROTECTED]: This probably should be rewritten to be outside of the drivers themselves so that it doesn't have to be done per-driver. I'm applying this to our XFree86 for now however until I've got time to investigate doing it more generically. diff -ur xc/programs/Xserver/hw/xfree86/drivers/ati.ORIG/r128_dri.c xc/programs/Xserver/hw/xfree86/drivers/ati/r128_dri.c --- xc/programs/Xserver/hw/xfree86/drivers/ati.ORIG/r128_dri.c 2002-10-20 02:48:27.0 +0900 +++ xc/programs/Xserver/hw/xfree86/drivers/ati/r128_dri.c 2002-10-20 03:05:24.0 +0900 @@ -54,15 +54,7 @@ #include "GL/glxtokens.h" #include "sarea.h" -/* ?? HACK - for now, put this here... */ -/* ?? Alpha - this may need to be a variable to handle UP1x00 vs TITAN */ -#if defined(__alpha__) -# define DRM_PAGE_SIZE 8192 -#elif defined(__ia64__) -# define DRM_PAGE_SIZE getpagesize() -#else -# define DRM_PAGE_SIZE 4096 -#endif +static size_t r128_drm_page_size; /* Initialize the visual configs that are supported by the hardware. These are combined with the visual configs that the indirect @@ -489,11 +481,11 @@ /* Initialize the CCE ring buffer data */ info->ringStart = info->agpOffset; -info->ringMapSize = info->ringSize*1024*1024 + DRM_PAGE_SIZE; +info->ringMapSize = info->ringSize*1024*1024 + r128_drm_page_size; info->ringSizeLog2QW = R128MinBits(info->ringSize*1024*1024/8) - 1; info->ringReadOffset = info->ringStart + info->ringMapSize; -info->ringReadMapSize = DRM_PAGE_SIZE; +info->ringReadMapSize = r128_drm_page_size; /* Reserve space for vertex/indirect buffers */ info->bufStart= info->ringReadOffset + info->ringReadMapSize; @@ -642,11 +634,11 @@ /* Initialize the CCE ring buffer data */ info->ringStart = info->agpOffset; -info->ringMapSize = info->ringSize*1024*1024 + DRM_PAGE_SIZE; +info->ringMapSize = info->ringSize*1024*1024 + r128_drm_page_size; info->ringSizeLog2QW = R128MinBits(info->ringSize*1024*1024/8) - 1; info->ringReadOffset = info->ringStart + info->ringMapSize; -info->ringReadMapSize = DRM_PAGE_SIZE; +info->ringReadMapSize = r128_drm_page_size; /* Reserve space for vertex/indirect buffers */ info->bufStart= info->ringReadOffset + info->ringReadMapSize; @@ -1003,6 +993,8 @@ break; } +r128_drm_page_size = getpagesize(); + /* Create the DRI data structure, and fill it in before calling the DRIScreenInit(). */ if (!(pDRIInfo = DRICreateInfoRec())) return FALSE; diff -ur xc/programs/Xserver/hw/xfree86/drivers/ati.ORIG/radeon_dri.c xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_dri.c --- xc/programs/Xserver/hw/xfree86/drivers/ati.ORIG/radeon_dri.c2002-10-20 02:48:27.0 +0900 +++ xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_dri.c 2002-10-20 03:04:18.0 +0900 @@ -56,15 +56,7 @@ #include "sarea.h" #include "radeon_sarea.h" -/* HACK - for now, put this here... */ -/* Alpha - this may need to be a variable to handle UP1x00 vs TITAN */ -#if defined(__alpha__) -# define DRM_PAGE_SIZE 8192 -#elif defined(__ia64__) -# define DRM_PAGE_SIZE getpagesize() -#else -# define DRM_PAGE_SIZE 4096 -#endif +static size_t radeon_drm_page_size; static Bool RADEONDRICloseFullScreen(ScreenPtr pScreen); @@ -774,11 +766,11 @@ /* Initialize the CP ring buffer data */ info->ringStart = info->agpOffset; -info->ringMapSize = info->ringSize*1024*1024 + DRM_PAGE_SIZE; +info->ringMapSize = info->ringSize*1024*1024 + radeon_drm_page_size; info->ringSizeLog2QW = RADEONMinBits(info->ringSize*1024*1024/8)-1; info->ringReadOffset = info->ringStart + info->ringMapSize; -info->ringReadMapSize = DRM_PAGE_SIZE; +info->ringReadMapSize = radeon_drm_page_size; /* Reserve space for vertex/indirect buffers */ info->bufStart= info->ringReadOffset + info->ringReadMapSize; @@ -900,11 +892,11 @@ /* Initialize the CCE ring buffer data */ info->ringStart = info->agpOffset; -info->ringMapSize = info->ringSize*1024*1024 + DRM_PAGE_SIZE; +info->ringMapSize = info->ringSize*1024*1024 + radeon_drm_page_size; info->ringSizeLog2QW = RADEONMinBits(info->ringSize*1024*1024/8)-1; info->ringReadOffset = info->ringStart + info->ringMapSize; -info->ringReadMapSize = DRM_PAGE_SIZE; +info->ringReadMapSize = radeon_drm_page_size; /* Reserve
patch for ia64 page size
Here is a patch for page size problems on ia64 and radeon. Patch by Chris Ahna: Fixes critical page size problems on ia64 architecture. See following URL for details: https://external-lists.valinux.com/archives/linux-ia64/2001-August/002037.html Comment by [EMAIL PROTECTED]: This probably should be rewritten to be outside of the drivers themselves so that it doesn't have to be done per-driver. I'm applying this to our XFree86 for now however until I've got time to investigate doing it more generically. diff -ur xc/programs/Xserver/hw/xfree86/drivers/ati.ORIG/r128_dri.c xc/programs/Xserver/hw/xfree86/drivers/ati/r128_dri.c --- xc/programs/Xserver/hw/xfree86/drivers/ati.ORIG/r128_dri.c 2002-10-20 02:48:27.0 +0900 +++ xc/programs/Xserver/hw/xfree86/drivers/ati/r128_dri.c 2002-10-20 03:05:24.0 +0900 @@ -54,15 +54,7 @@ #include "GL/glxtokens.h" #include "sarea.h" -/* ?? HACK - for now, put this here... */ -/* ?? Alpha - this may need to be a variable to handle UP1x00 vs TITAN */ -#if defined(__alpha__) -# define DRM_PAGE_SIZE 8192 -#elif defined(__ia64__) -# define DRM_PAGE_SIZE getpagesize() -#else -# define DRM_PAGE_SIZE 4096 -#endif +static size_t r128_drm_page_size; /* Initialize the visual configs that are supported by the hardware. These are combined with the visual configs that the indirect @@ -489,11 +481,11 @@ /* Initialize the CCE ring buffer data */ info->ringStart = info->agpOffset; -info->ringMapSize = info->ringSize*1024*1024 + DRM_PAGE_SIZE; +info->ringMapSize = info->ringSize*1024*1024 + r128_drm_page_size; info->ringSizeLog2QW = R128MinBits(info->ringSize*1024*1024/8) - 1; info->ringReadOffset = info->ringStart + info->ringMapSize; -info->ringReadMapSize = DRM_PAGE_SIZE; +info->ringReadMapSize = r128_drm_page_size; /* Reserve space for vertex/indirect buffers */ info->bufStart= info->ringReadOffset + info->ringReadMapSize; @@ -642,11 +634,11 @@ /* Initialize the CCE ring buffer data */ info->ringStart = info->agpOffset; -info->ringMapSize = info->ringSize*1024*1024 + DRM_PAGE_SIZE; +info->ringMapSize = info->ringSize*1024*1024 + r128_drm_page_size; info->ringSizeLog2QW = R128MinBits(info->ringSize*1024*1024/8) - 1; info->ringReadOffset = info->ringStart + info->ringMapSize; -info->ringReadMapSize = DRM_PAGE_SIZE; +info->ringReadMapSize = r128_drm_page_size; /* Reserve space for vertex/indirect buffers */ info->bufStart= info->ringReadOffset + info->ringReadMapSize; @@ -1003,6 +993,8 @@ break; } +r128_drm_page_size = getpagesize(); + /* Create the DRI data structure, and fill it in before calling the DRIScreenInit(). */ if (!(pDRIInfo = DRICreateInfoRec())) return FALSE; diff -ur xc/programs/Xserver/hw/xfree86/drivers/ati.ORIG/radeon_dri.c xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_dri.c --- xc/programs/Xserver/hw/xfree86/drivers/ati.ORIG/radeon_dri.c2002-10-20 02:48:27.0 +0900 +++ xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_dri.c 2002-10-20 03:04:18.0 +0900 @@ -56,15 +56,7 @@ #include "sarea.h" #include "radeon_sarea.h" -/* HACK - for now, put this here... */ -/* Alpha - this may need to be a variable to handle UP1x00 vs TITAN */ -#if defined(__alpha__) -# define DRM_PAGE_SIZE 8192 -#elif defined(__ia64__) -# define DRM_PAGE_SIZE getpagesize() -#else -# define DRM_PAGE_SIZE 4096 -#endif +static size_t radeon_drm_page_size; static Bool RADEONDRICloseFullScreen(ScreenPtr pScreen); @@ -774,11 +766,11 @@ /* Initialize the CP ring buffer data */ info->ringStart = info->agpOffset; -info->ringMapSize = info->ringSize*1024*1024 + DRM_PAGE_SIZE; +info->ringMapSize = info->ringSize*1024*1024 + radeon_drm_page_size; info->ringSizeLog2QW = RADEONMinBits(info->ringSize*1024*1024/8)-1; info->ringReadOffset = info->ringStart + info->ringMapSize; -info->ringReadMapSize = DRM_PAGE_SIZE; +info->ringReadMapSize = radeon_drm_page_size; /* Reserve space for vertex/indirect buffers */ info->bufStart= info->ringReadOffset + info->ringReadMapSize; @@ -900,11 +892,11 @@ /* Initialize the CCE ring buffer data */ info->ringStart = info->agpOffset; -info->ringMapSize = info->ringSize*1024*1024 + DRM_PAGE_SIZE; +info->ringMapSize = info->ringSize*1024*1024 + radeon_drm_page_size; info->ringSizeLog2QW = RADEONMinBits(info->ringSize*1024*1024/8)-1; info->ringReadOffset = info->ringStart + info->ringMapSize; -info->ringReadMapSize = DRM_PAGE_SIZE; +info->ringReadMapSize = radeon_drm_page_size; /* Reserve
Re: patch for ia64 page size
Jakub Jelinek wrote: On Sun, Aug 10, 2003 at 07:06:58PM -0500, Warren Turkal wrote: @@ -1003,6 +993,8 @@ break; } +r128_drm_page_size = getpagesize(); + sysconf (_SC_PAGESIZE) is the standardized way of querying page size. I seem to recall some discussion about this a few months ago. There are some portability issues with both getpagesize and sysconf(_SC_PAGESIZE). Because of that, XFree86 has a wrapper function called xf86getpagesize. There also seems to be a #define that aliases getpagesize to xf86getpagesize, so I'm not sure if the wrapper should be used or if getpagesize should be used. Either way, I'm sure that sysconf(_SG_PAGESIZE) should *not* be used directly. ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
Re: patch for ia64 page size
You might want to create a bug in bugzilla (bugs.xfree86.org) and attach the patches there. that way people can comment on the patches and the committers usually post a comment there if they decide to include it. Alex --- Warren Turkal <[EMAIL PROTECTED]> wrote: > BTW, if I submit a patch to patches@, will I get a notification of > acceptance or rejection? > > Warren Turkal __ Do you Yahoo!? Yahoo! SiteBuilder - Free, easy-to-use web site design software http://sitebuilder.yahoo.com ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
Re: patch for ia64 page size
On Mon, 11 Aug 2003, Jakub Jelinek wrote: > On Sun, Aug 10, 2003 at 07:06:58PM -0500, Warren Turkal wrote: > > @@ -1003,6 +993,8 @@ > > break; > > } > > > > +r128_drm_page_size = getpagesize(); > > + > sysconf (_SC_PAGESIZE) > is the standardized way of querying page size. ... but non-portable to pre-standard systems. getpagesize() is provided on all platforms by the loader server's core binary. It is also available in the static server for systems that don't provide it. The one in libc is used if provided. Marc. +--+---+ | Marc Aurele La France | work: 1-780-492-9310 | | Computing and Network Services | fax:1-780-492-1729 | | 352 General Services Building | email: [EMAIL PROTECTED] | | University of Alberta +---+ | Edmonton, Alberta | | | T6G 2H1 | Standard disclaimers apply| | CANADA | | +--+---+ XFree86 Core Team member. ATI driver and X server internals. ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
Re: patch for ia64 page size
On Mon, Aug 11, 2003 at 07:06:01AM +0200, Jakub Jelinek wrote: >On Sun, Aug 10, 2003 at 07:06:58PM -0500, Warren Turkal wrote: >> @@ -1003,6 +993,8 @@ >> break; >> } >> >> +r128_drm_page_size = getpagesize(); >> + > >sysconf (_SC_PAGESIZE) >is the standardized way of querying page size. getpagesize() as invoked from a driver is aliased to xf86getpagesize(), which does this: #if defined(linux) #define HAS_SC_PAGESIZE #define HAS_GETPAGESIZE #elif defined(CSRG_BASED) #define HAS_GETPAGESIZE #elif defined(DGUX) #define HAS_GETPAGESIZE #elif defined(sun) && !defined(SVR4) #define HAS_GETPAGESIZE #endif #ifdef XNO_SYSCONF #undef _SC_PAGESIZE #endif ... int xf86getpagesize() { static int pagesize = -1; if (pagesize != -1) return pagesize; #if defined(_SC_PAGESIZE) || defined(HAS_SC_PAGESIZE) pagesize = sysconf(_SC_PAGESIZE); #endif #ifdef _SC_PAGE_SIZE if (pagesize == -1) pagesize = sysconf(_SC_PAGE_SIZE); #endif #ifdef HAS_GETPAGESIZE if (pagesize == -1) pagesize = getpagesize(); #endif #ifdef PAGE_SIZE if (pagesize == -1) pagesize = PAGE_SIZE; #endif if (pagesize == -1) FatalError("xf86getpagesize: Cannot determine page size\n"); return pagesize; } David -- David Dawes Founder/committer/developer The XFree86 Project www.XFree86.org/~dawes ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
Re: patch for ia64 page size
On Sun, Aug 10, 2003 at 07:06:58PM -0500, Warren Turkal wrote: > @@ -1003,6 +993,8 @@ > break; > } > > +r128_drm_page_size = getpagesize(); > + sysconf (_SC_PAGESIZE) is the standardized way of querying page size. Jakub ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
Re: patch for ia64 page size
On Sun, Aug 10, 2003 at 07:06:58PM -0500, Warren Turkal wrote: >Here is a patch for page size problems on ia64 and radeon. At least some of the patches you're sending (this one in particular) have been previously submitted and reviewed. Specifically, this patch makes no actual change for ia64, the architecture claimed to be the problem. >Patch by Chris Ahna: > >Fixes critical page size problems on ia64 architecture. See following URL >for >details: > >https://external-lists.valinux.com/archives/linux-ia64/2001-August/002037.html > >Comment by [EMAIL PROTECTED]: > >This probably should be rewritten to be outside of the drivers themselves so >that it doesn't have to be done per-driver. I'm applying this to our >XFree86 for now however until I've got time to investigate doing it more >generically. As I said when I reviewed this originally, getpagesize(), which is aliased to xf86getpagesize(), should reliably find the page size on all architectures, so this comment is redundant. DRM_PAGE_SIZE should simply be defined to getpagesize(), providing that is consistent with what the kernel interfaces expect. David -- David Dawes Founder/committer/developer The XFree86 Project www.XFree86.org/~dawes ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
Re: patch for ia64 page size
Jakub Jelinek wrote: > On Sun, Aug 10, 2003 at 07:06:58PM -0500, Warren Turkal wrote: >> @@ -1003,6 +993,8 @@ >> break; >> } >> >> +r128_drm_page_size = getpagesize(); >> + > > sysconf (_SC_PAGESIZE) > is the standardized way of querying page size. > > Jakub I didn't write this patch. Are you saying that the line with r128_drm_page_size should be the following: r128_drm_page_size = sysconf(_SC_PAGESIZE) If so, I will change it, but I cannot test it as I don't have access to ia64 hardware. Also, I just wanna remind everyone that these patches I am sending come from the debian packages. I wrote none of them myself. I am just trying to make sure that all fixes that should go upstream do. BTW, if I submit a patch to patches@, will I get a notification of acceptance or rejection? Warren Turkal -- Treasurer, GOLUM, Inc. http://www.golum.org ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
Re: patch for ia64 page size
On Sun, 10 Aug 2003, Warren Turkal wrote: >Date: Sun, 10 Aug 2003 19:06:58 -0500 >From: Warren Turkal <[EMAIL PROTECTED]> >To: [EMAIL PROTECTED] >Reply-To: [EMAIL PROTECTED] >Content-Type: text/plain; charset=us-ascii >Subject: patch for ia64 page size > >Here is a patch for page size problems on ia64 and radeon. > >Patch by Chris Ahna: > >Fixes critical page size problems on ia64 architecture. See following URL >for >details: > >https://external-lists.valinux.com/archives/linux-ia64/2001-August/002037.html > >Comment by [EMAIL PROTECTED]: Why are you submitting _my_ patch, without even asking me about it? If you had, I'd have told you that that patch is an ugly mess, not suitable for upstream submission for numerous reasons. It works, but it is far from clean and ready for inclusion in CVS. Before submitting my patches upstream, please contact me first to find out what my own plans are. I generally send patches upstream myself personally when I feel they are ready to be included upstream. If I haven't, it is a good idea to ask me why not first. I can save you some headaches. -- Mike A. Harris ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
Re: patch for ia64 page size
On Mon, 11 Aug 2003, Alex Deucher wrote: >Date: Mon, 11 Aug 2003 06:25:21 -0700 (PDT) >From: Alex Deucher <[EMAIL PROTECTED]> >To: [EMAIL PROTECTED] >Reply-To: [EMAIL PROTECTED] >Content-Type: text/plain; charset=us-ascii >Subject: Re: patch for ia64 page size > >You might want to create a bug in bugzilla (bugs.xfree86.org) and >attach the patches there. that way people can comment on the patches >and the committers usually post a comment there if they decide to >include it. No, please don't. Those pagesize patches are ugly hacks currently, and are not suitable for XFree86.org. Don't waste their time please. -- Mike A. Harris ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
Re: patch for ia64 page size
On Mon, 11 Aug 2003, Ian Romanick wrote: >>>@@ -1003,6 +993,8 @@ >>>break; >>> } >>> >>>+r128_drm_page_size = getpagesize(); >>>+ >> >> sysconf (_SC_PAGESIZE) >> is the standardized way of querying page size. > >I seem to recall some discussion about this a few months ago. There are >some portability issues with both getpagesize and sysconf(_SC_PAGESIZE). > Because of that, XFree86 has a wrapper function called >xf86getpagesize. There also seems to be a #define that aliases >getpagesize to xf86getpagesize, so I'm not sure if the wrapper should be >used or if getpagesize should be used. Either way, I'm sure that >sysconf(_SG_PAGESIZE) should *not* be used directly. Agreed. Both getpagesize() and sysconf have portability issues. One on older systems, the other on newer systems. I can't remember the specific, but HPUX had an issue with one I recall. I'd have to hunt down the manual I found that indicated this however. This should be dealt with in a very nice way by xf86getpagesize() more or less. There are some fixes that need to happen, at least from 4.3.0 codebase (not sure about current CVS head), but they need to be done in a clean manner. I intend to go through everywhere in the server and clean up anything I find at some point, and then send a proper patch in to fix it all, unless someone beats me to it. The mailing list archives have comments from other developers from the last time this topic came up. -- Mike A. Harris ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel
Re: patch for ia64 page size
Mike A. Harris wrote: > On Sun, 10 Aug 2003, Warren Turkal wrote: > >>Date: Sun, 10 Aug 2003 19:06:58 -0500 >>From: Warren Turkal <[EMAIL PROTECTED]> >>To: [EMAIL PROTECTED] >>Reply-To: [EMAIL PROTECTED] >>Content-Type: text/plain; charset=us-ascii >>Subject: patch for ia64 page size >> >>Here is a patch for page size problems on ia64 and radeon. >> >>Patch by Chris Ahna: >> >>Fixes critical page size problems on ia64 architecture. See following URL >>for >>details: >> >>https://external-lists.valinux.com/archives/linux-ia64/2001-August/002037.html >> >>Comment by [EMAIL PROTECTED]: > > Why are you submitting _my_ patch, without even asking me about > it? If you had, I'd have told you that that patch is an ugly > mess, not suitable for upstream submission for numerous reasons. > It works, but it is far from clean and ready for inclusion in > CVS. > > Before submitting my patches upstream, please contact me first to > find out what my own plans are. I generally send patches > upstream myself personally when I feel they are ready to be > included upstream. If I haven't, it is a good idea to ask me why > not first. I can save you some headaches. > > These patches are included in the debian packages for X. I was trying to make sure that the stuff got included in X so that other distributions could take advantage of it. I simply tried to make it apply and then sent it to the list in hopes that more knowledgable developers would let me know its status or include it if they chose. I was just trying to help. Warren Turkal -- Treasurer, GOLUM, Inc. http://www.golum.org ___ Devel mailing list [EMAIL PROTECTED] http://XFree86.Org/mailman/listinfo/devel