Re: ISM shared memory on solaris - Mailing list pgsql-bugs

From Tom Lane
Subject Re: ISM shared memory on solaris
Date
Msg-id 853.1067278484@sss.pgh.pa.us
Whole thread Raw
In response to Re: ISM shared memory on solaris  (Bruce Momjian <pgman@candle.pha.pa.us>)
Responses Re: ISM shared memory on solaris  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-bugs
Bruce Momjian <pgman@candle.pha.pa.us> writes:

> ! #if defined(sun) && defined(__sparc__)
>       /* use intimate shared memory on SPARC Solaris */
>       memAddress = shmat(shmid, 0, SHM_SHARE_MMU);
>   #else

I think this is going in the wrong direction.  Why isn't the code just

#if defined(SHM_SHARE_MMU)
    /* use intimate shared memory on Solaris */
    memAddress = shmat(shmid, 0, SHM_SHARE_MMU);
#else

If the symbol is available I think we probably want to use it.  It is an
O/S issue, not a hardware issue, and so the test on __sparc__ seems
quite wrongheaded ...

            regards, tom lane

pgsql-bugs by date:

Previous
From: Tom Lane
Date:
Subject: Re: PostgreSQL Patch: Test-and-set routine for HP-UX (IA-64)
Next
From: Bruce Momjian
Date:
Subject: Re: ISM shared memory on solaris