| From: | Claudio Natoli <claudio(dot)natoli(at)memetrics(dot)com> |
|---|---|
| To: | 'Tom Lane ' <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Claudio Natoli <claudio(dot)natoli(at)memetrics(dot)com> |
| Cc: | "''''pgsql-patches(at)postgresql(dot)org' ' ' '" <pgsql-patches(at)postgresql(dot)org> |
| Subject: | Re: APC/socket fix (final?) |
| Date: | 2004-03-26 01:29:14 |
| Message-ID: | A02DEC4D1073D611BAE8525405FCCE2B55F3BE@harris.memetrics.local |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-patches |
> Ugh. Is there a way we can insert a wrapper layer without modifying the
> call sites? I'm thinking of some kind of macro hack, say
> [snip]
Sure. Think we've even done this before (also, prevents developers needing
to remember to use pg_*).
The reason I think it was avoided for select(), in preference for a thread
to invoke the socket op during the signal/APC, was a fear that perhaps the
Windows Sockets internals could get mashed. AFAICS, the discussion Magnus
had with the Microsoft guys (and, from memory, those I've had with Magnus
off-list) suggests this isn't true. If mashing the internals is still a
possibility, then clearly the patch I've submitted might do more harm than
good.
(Magnus, can you confirm?)
If so, I'll submit a patch for select/recv/send over the weekend, which will
also remove the recent fixes for pgstat.
Cheers,
Claudio
---
Certain disclaimers and policies apply to all email sent from Memetrics.
For the full text of these disclaimers and policies see
<a
href="http://www.memetrics.com/emailpolicy.html">http://www.memetrics.com/em
ailpolicy.html</a>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alvaro Herrera | 2004-03-26 02:09:17 | Re: installdir patch for win32 |
| Previous Message | Tom Lane | 2004-03-26 01:07:42 | Re: APC/socket fix (final?) |