Hi David, I did some testing with this patch. It looks OK. Do you plan to commit the patch into the git tree?
Thanks Gang > -----Original Message----- > From: David Teigland [mailto:teigl...@redhat.com] > Sent: 2020年4月27日 23:04 > To: Gang He <g...@suse.com> > Cc: LVM general discussion and development <linux-lvm@redhat.com> > Subject: Re: [linux-lvm] can we change cluster_ringid_seq in libdlm from > uint32_t to uint64_t? > > On Sun, Apr 26, 2020 at 07:24:33AM +0000, Gang He wrote: > > Hello List, > > > > In libdlm code, cluster_ringid_seq variable is defined with uint32_t > > in dlm_controld/dlm_daemon.h, the corosync API returns uinit64_t ring_id, > in the current code, we use type cast to get the low-32bit ring-id. > > But, in some case, the corosync returns a huge ring-id (greater than 32bit), > the DLM daemon does not work normally (looks stuck). > > Then, I want to know if we can change cluster_ringid_seq in libdlm from > uint32_t to uint64_t? > > That looks ok, please try the attached patch. > Dave _______________________________________________ linux-lvm mailing list linux-lvm@redhat.com https://www.redhat.com/mailman/listinfo/linux-lvm read the LVM HOW-TO at http://tldp.org/HOWTO/LVM-HOWTO/
