Thread

Re: Proposal: Add a callback data parameter to GetNamedDSMSegment

Sami Imseih <samimseih@gmail.com> — 2025-12-12T21:56:07Z

> > [...]
> > -       LWLockInitialize(&dsm->lck, LWLockNewTrancheId("test_dsm_registry"));
> > +       LWLockInitialize(&dsm->lck, LWLockNewTrancheId((char *) arg));
> >         dsm->val = 0;
> > [...]
> >
> > instead of creating a new test? For the other GetNamedDSMSegment calls,
> > I'll pass NULL to the void * and hard code the tranche name in the init
> > callback.
>
> I think we should verify the pointer value more directly.  For example, we
> could pass something like (uintptr_t) 0x12345 via the callback argument and
> then verify it's the same in the callback.

This is better than what I had earlier where I was testing that a string
was passed correctly. See v6.

--
Sami Imseih
Amazon Web Services(AWS)