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:teigland@xxxxxxxxxx] > Sent: 2020年4月27日 23:04 > To: Gang He <GHe@xxxxxxxx> > Cc: LVM general discussion and development <linux-lvm@xxxxxxxxxx> > Subject: Re: 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@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/linux-lvm read the LVM HOW-TO at http://tldp.org/HOWTO/LVM-HOWTO/
