On Thu, Feb 11, 2016 at 5:43 PM, Bob Peterson <rpete...@redhat.com> wrote: > Hi Steve, > > ----- Original Message ----- >> Might be clearer to move the test for CF_IS_OTHERCON outside of these >> functions and into the callers? >> >> Otherwise these patches look like a good set of clean ups, >> >> Steve. > > Good idea. Here's a replacement patch that implements your suggestion. > > Regards, > > Bob Peterson > Red Hat File Systems > --- > DLM: save / restore all socket callbacks > > Before this patch, DLM was saving off the original error report > callback before setting its own, but it never restored it. Instead, > we should be saving off all four socket callbacks before changing > them, and then restore them once we're done. > > Signed-off-by: Bob Peterson <rpete...@redhat.com>
Reviewed-by: Andreas Gruenbacher <agrue...@redhat.com>
