Re: [pgsql-patches] O_DIRECT support for Windows - Mailing list pgsql-patches

From Magnus Hagander
Subject Re: [pgsql-patches] O_DIRECT support for Windows
Date
Msg-id 20070413102733.GM31884@svr2.hagander.net
Whole thread Raw
In response to Re: [pgsql-patches] O_DIRECT support for Windows  (Bruce Momjian <bruce@momjian.us>)
List pgsql-patches
I've done some further looking aruond at this, and I've been unable to find
any references to disk systems with sector size > 8192 bytes (which is what
the alignment of the buffers per XLOG_BLCKSZ, at leastby default).

So I'll commit this fairly simple patch, and we'll revert it or add runtime
checks later if we find out that there exist such systems somewhere..

//Magnus


On Mon, Apr 02, 2007 at 04:39:00PM -0400, Bruce Momjian wrote:
>
> Your patch has been added to the PostgreSQL unapplied patches list at:
>
>     http://momjian.postgresql.org/cgi-bin/pgpatches
>
> It will be applied as soon as one of the PostgreSQL committers reviews
> and approves it.
>
> ---------------------------------------------------------------------------
>
>
> ITAGAKI Takahiro wrote:
> > The attached is a patch to define O_DIRECT by ourselves on Windows,
> > and to map O_DIRECT to FILE_FLAG_NO_BUFFERING.
> >
> > There will be a consistency in our support between Windows and other OSes
> > that have O_DIRECT. Also, there is the following comment that says, I read,
> > we should do so.
> > | handle other flags? (eg FILE_FLAG_NO_BUFFERING/FILE_FLAG_WRITE_THROUGH)
> >
> > Is this worth doing? Do we need more performance reports for the change?
> >
> > Regards,
> > ---
> > ITAGAKI Takahiro
> > NTT Open Source Software Center
>
> [ Attachment, skipping... ]
>
> >
> > ---------------------------(end of broadcast)---------------------------
> > TIP 2: Don't 'kill -9' the postmaster
>
> --
>   Bruce Momjian  <bruce@momjian.us>          http://momjian.us
>   EnterpriseDB                               http://www.enterprisedb.com
>
>   + If your life is a hard drive, Christ can be your backup. +
>
> ---------------------------(end of broadcast)---------------------------
> TIP 9: In versions below 8.0, the planner will ignore your desire to
>        choose an index scan if your joining column's datatypes do not
>        match

pgsql-patches by date:

Previous
From: "Zeugswetter Andreas ADI SD"
Date:
Subject: Re: [HACKERS] Full page writes improvement, code update
Next
From: "Simon Riggs"
Date:
Subject: Re: Reviewers Guide to DeferredTransactions/TransactionGuarantee