On Mon, Feb 22, 2010 at 06:00:21PM +0100, Jan Friesse wrote:
> +struct cpg_ring_id {
> +     uint32_t nodeid;
> +     uint64_t seq;
> +};

What do you think about combining this patch with the other patch that
adds cpg_ringid_get()?  It's troublesome to combine the two patches to
test.

> +typedef void (*cpg_totem_confchg_fn_t) (
> +     cpg_handle_t handle,
> +     struct cpg_ring_id ring_id,
> +     uint32_t member_list_entries,
> +     const uint32_t *member_list);

I've started working on the code to use this, and it might be nice if the
parameters matched the normal confchg parameters as closely as possible,
i.e. include cpg_name, and use cpg_address instead of uint32_t.

Dave

_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to