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

From Bruce Momjian
Subject Re: [pgsql-patches] O_DIRECT support for Windows
Date
Msg-id 200703272008.l2RK8XJ20298@momjian.us
Whole thread Raw
In response to Re: O_DIRECT support for Windows  (Magnus Hagander <magnus@hagander.net>)
Responses Re: [pgsql-patches] O_DIRECT support for Windows  (Magnus Hagander <magnus@hagander.net>)
List pgsql-patches
Magnus, where are on this?

---------------------------------------------------------------------------

Magnus Hagander wrote:
> We're ok with the alignment issues provided the is code added to reject
> O_DIRECT if the sector size is too large.
>
> We also said we need to see some performance numbers on the effect of
> the patch before it goes in.
>
> //Magnus
>
>
> Bruce Momjian wrote:
> > So, do we want this patch?  Are we OK on WIN32 alignment issues?
> >
> > ---------------------------------------------------------------------------
> >
> > 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
> >
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 7: You can help support the PostgreSQL project by donating at
>
>                 http://www.postgresql.org/about/donate

--
  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. +

pgsql-patches by date:

Previous
From: Heikki Linnakangas
Date:
Subject: Re: Numeric patch to add special-case representations for < 8 bytes
Next
From: "Marko Kreen"
Date:
Subject: Re: DEALLOCATE ALL