Re: O_DIRECT support for Windows

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: ITAGAKI Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp>, pgsql-patches(at)postgresql(dot)org
Subject: Re: O_DIRECT support for Windows
Date: 2007-01-20 11:17:04
Message-ID: 45B1FA30.9080307@hagander.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

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
>

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2007-01-20 12:18:15 Re: Planning aggregates which require sorted or distinct
Previous Message Simon Riggs 2007-01-20 11:06:23 Re: Planning aggregates which require sorted or distinct

Browse pgsql-patches by date

  From Date Subject
Next Message Magnus Hagander 2007-01-20 11:19:59 Re: guid/uuid datatype
Previous Message Pavel Stehule 2007-01-20 09:52:54 Re: scrollable cursor sup. for SPI