Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Linux-HA: Dev

Re: [Linux-HA] socket of lrmd

 

 

Linux-HA dev RSS feed   Index | Next | Previous | View Threaded


florian.haas at linbit

Dec 2, 2009, 1:25 AM

Post #1 of 2 (565 views)
Permalink
Re: [Linux-HA] socket of lrmd

[Moving this to the -dev list]

Dejan,

Please revisit the original thread too; Lars Ellenberg had an
alternative suggestion for fixing this issue.

Is there a Bugzilla entry for this?

Cheers,
Florian

On 2009-12-02 05:34, Junko IKEDA wrote:
> Hi Dejan,
>
> This is an old issue,
> http://www.gossamer-threads.com/lists/linuxha/users/56449
> but I remember this from the release plan announcement of Heartbeat 3.0.2.
>
> I have done the test for the following your patch,
> it seemed there was no problem.
> Please apply this to the new release.
>
> On Thu, 02 Jul 2009 22:04:06 +0900, Dejan Muhamedagic
> <dejanmm [at] fastmail> wrote:
>
>> Right. See lib/lrm/racommon.c:closefiles(). I must say I can't
>> remember why exactly was the connection to syslogd/ha_logd
>> needed. There has been a problem with stonithd, but now this code
>> is not in that execution path anymore. Perhaps you can test this
>> patch:
>>
>> diff -r 731f8f7b5450 lib/lrm/racommon.c
>> --- a/lib/lrm/racommon.c Tue Jun 30 12:02:16 2009 +0200
>> +++ b/lib/lrm/racommon.c Thu Jul 02 15:02:55 2009 +0200
>> @@ -145,7 +145,7 @@
>> /* close all descriptors except stdin/out/err and
>> * channels to logd */
>> for (fd = getdtablesize(); fd > STDERR_FILENO; fd--) {
>> - if (!cl_log_is_logd_fd(fd))
>> + /*if (!cl_log_is_logd_fd(fd))*/
>> close(fd);
>> }
>> }
>
> Thanks,
> Junko
Attachments: signature.asc (0.25 KB)


dejanmm at fastmail

Dec 11, 2009, 5:00 AM

Post #2 of 2 (505 views)
Permalink
Re: [Linux-HA] socket of lrmd [In reply to]

Hi Florian,

On Wed, Dec 02, 2009 at 10:25:00AM +0100, Florian Haas wrote:
> [Moving this to the -dev list]
>
> Dejan,
>
> Please revisit the original thread too; Lars Ellenberg had an
> alternative suggestion for fixing this issue.
>
> Is there a Bugzilla entry for this?

Yes. I just saw it last night. And missed these messages.

Thanks,

Dejan

> Cheers,
> Florian
>
> On 2009-12-02 05:34, Junko IKEDA wrote:
> > Hi Dejan,
> >
> > This is an old issue,
> > http://www.gossamer-threads.com/lists/linuxha/users/56449
> > but I remember this from the release plan announcement of Heartbeat 3.0.2.
> >
> > I have done the test for the following your patch,
> > it seemed there was no problem.
> > Please apply this to the new release.
> >
> > On Thu, 02 Jul 2009 22:04:06 +0900, Dejan Muhamedagic
> > <dejanmm [at] fastmail> wrote:
> >
> >> Right. See lib/lrm/racommon.c:closefiles(). I must say I can't
> >> remember why exactly was the connection to syslogd/ha_logd
> >> needed. There has been a problem with stonithd, but now this code
> >> is not in that execution path anymore. Perhaps you can test this
> >> patch:
> >>
> >> diff -r 731f8f7b5450 lib/lrm/racommon.c
> >> --- a/lib/lrm/racommon.c Tue Jun 30 12:02:16 2009 +0200
> >> +++ b/lib/lrm/racommon.c Thu Jul 02 15:02:55 2009 +0200
> >> @@ -145,7 +145,7 @@
> >> /* close all descriptors except stdin/out/err and
> >> * channels to logd */
> >> for (fd = getdtablesize(); fd > STDERR_FILENO; fd--) {
> >> - if (!cl_log_is_logd_fd(fd))
> >> + /*if (!cl_log_is_logd_fd(fd))*/
> >> close(fd);
> >> }
> >> }
> >
> > Thanks,
> > Junko
>



> _______________________________________________________
> Linux-HA-Dev: Linux-HA-Dev [at] lists
> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> Home Page: http://linux-ha.org/

_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev [at] lists
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

Linux-HA dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.