From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Dilip Kumar <dilipbalaut(at)gmail(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-committers <pgsql-committers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: pgsql: pgstat: Bring up pgstat in BaseInit() to fix uninitialized use o |
Date: | 2021-08-13 09:09:35 |
Message-ID: | CAA4eK1KMxJMEdDVFQZ7qn9ZgjY=anUf9X3Jbun7q7Ph=azg+ng@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Thu, Aug 12, 2021 at 6:24 PM Andres Freund <andres(at)anarazel(dot)de> wrote:
>
> On 2021-08-12 05:48:19 -0700, Andres Freund wrote:
> > I think SharedFileSetInit() needs a comment explaining that it needs to be
> > called in a process-lifetime memory context if used without dsm
> > segments. Because otherwise SharedFileSetDeleteOnProcExit() will access
> > already freed memory (both for filesetlist and the SharedFileSet itself).
>
> Oh. And I think it's not ok that SharedFileSetDeleteAll() unconditionally does
> SharedFileSetUnregister(). SharedFileSetUnregister() asserts out if there's no
> match, but DSM based sets are never entered into filesetlist. So one cannot
> have a non-DSM and DSM set coexisting. Which seems surprising.
>
Oops, it should be allowed to have both non-DSM and DSM set
coexisting. I think we can remove Assert from
SharedFileSetUnregister(). The other way could be to pass a parameter
to SharedFileSetDeleteAll() to tell whether to unregister or not.
--
With Regards,
Amit Kapila.
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2021-08-13 12:47:09 | pgsql: Fix typo. |
Previous Message | Amit Kapila | 2021-08-13 09:08:37 | Re: pgsql: pgstat: Bring up pgstat in BaseInit() to fix uninitialized use o |
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2021-08-13 09:17:51 | Re: [bug] Logical Decoding of relation rewrite with toast does not reset toast_hash |
Previous Message | Amit Kapila | 2021-08-13 09:08:37 | Re: pgsql: pgstat: Bring up pgstat in BaseInit() to fix uninitialized use o |