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 460980D0.6070703@hagander.net
Whole thread Raw
In response to Re: [pgsql-patches] O_DIRECT support for Windows  (Bruce Momjian <bruce@momjian.us>)
List pgsql-patches
IIRC, we're still waiting for performance numbers showing there exists a
win from this patch.

//Magnus

Bruce Momjian wrote:
> 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
>


pgsql-patches by date:

Previous
From: Holger Schurig
Date:
Subject: [PATCH] add CLUSTER table ORDER BY index
Next
From: Alvaro Herrera
Date:
Subject: autovacuum: recheck logic