View previous topic :: View next topic |
Author |
Message |
rvdp n00b
Joined: 27 May 2002 Posts: 11
|
Posted: Mon Jul 01, 2002 6:32 pm Post subject: CARDMGR -f |
|
|
Sorry to be rude, please enlighten me.
When is -f gona be put back in /etc/conf.d/pcmcia??
Sooo many people I have helped by telling them this 'trick'
Also, even though having no -f completley braeks cardmgr, and it should be set, the latest .34 version has no problem with/without it.
1.3 at least. Pleeeease
Patrick. _________________ Your pc will not crash today |
|
Back to top |
|
|
chadh Moderator
Joined: 21 Apr 2002 Posts: 137 Location: Atlanta, GA
|
Posted: Mon Jul 01, 2002 10:58 pm Post subject: |
|
|
Well, originally we had it in b/c it caused problems with people who had both the pcmcia init script *and* the net.eth? scripts in their runlevel. Since we changed the scheme to let cardmgr run the net.eth? init scripts, that problem should have gone away. cardmgr works fine for me without the -f. What exactly is the problem when that option is not there? _________________ Chad |
|
Back to top |
|
|
rvdp n00b
Joined: 27 May 2002 Posts: 11
|
Posted: Tue Jul 02, 2002 2:12 am Post subject: |
|
|
A bug in cardmgr versions less than 3.1.34 exist. Acoording to sourceforge the problem is fixed in .34.
The bug is that /var/run/stab is deleted if cardmgr is not run with -f.
As I saw in bugzilla, -f was removed from gentoo, but a later coment by drobbins said it was put back. duno wat happened there.
Having no -f dosent 100% save the net.eth? in init problem, I was chating to some one who had no -f and net.eth? in init. It didnt work.
But no -f and no net.eth? in the init is fine...untill users want to eject etc.
Often they will not see the 'cant find stab file' error, and will wonder why net.eth0 is still up and all there netmounts are hung....
Now, the bug is fixed in .34, but -f should be used anyway, IMO.
Hope this helps
Patrick. _________________ Your pc will not crash today |
|
Back to top |
|
|
chadh Moderator
Joined: 21 Apr 2002 Posts: 137 Location: Atlanta, GA
|
Posted: Tue Jul 02, 2002 3:14 am Post subject: |
|
|
Okay pcmcia-cs-3.1.34 is in portage. I also added the -f flag back. The ebuilds are masked for now, but feel free to try them and let me know how they work. _________________ Chad |
|
Back to top |
|
|
rvdp n00b
Joined: 27 May 2002 Posts: 11
|
Posted: Tue Jul 02, 2002 6:33 am Post subject: |
|
|
Hey chad, real sorry bout this.
The following "atexit" are additional to .33, thus an attempt to fix unlinking of the stab file.
They foo bared .34
if (delay_fork) {
atexit(NULL);
fork_now();
atexit(&done);
write_pid();
atexit(NULL) seg faults!
.34 ChangeLog:
"Fixed bug with premature file cleanup in cardmgr when using "-f"
(delayed fork) option."
Not quite..
Till next time.
Patrick. _________________ Your pc will not crash today |
|
Back to top |
|
|
chadh Moderator
Joined: 21 Apr 2002 Posts: 137 Location: Atlanta, GA
|
Posted: Tue Jul 02, 2002 12:42 pm Post subject: |
|
|
Have you tried it, or did you just come up with that from an audit of the code? I confirm all that, but I want to make sure that cardmgr actually crashes on exit. _________________ Chad |
|
Back to top |
|
|
rvdp n00b
Joined: 27 May 2002 Posts: 11
|
Posted: Wed Jul 03, 2002 4:17 am Post subject: |
|
|
Yup, segfaults on exit.
cardmgr continues to run, it forks and the parent crashes.
Efectivley the only problem is that pcmcia fails to start, but besides that the pcmcia card is up and running. _________________ Your pc will not crash today |
|
Back to top |
|
|
chadh Moderator
Joined: 21 Apr 2002 Posts: 137 Location: Atlanta, GA
|
Posted: Wed Jul 03, 2002 4:36 am Post subject: |
|
|
Well, I submitted a bug report, and I threw together a patch that removes that atexit(NULL) call, but I don't know if that is the proper fix.
It is masked as pcmcia-cs-3.1.34-r2.ebuild _________________ Chad |
|
Back to top |
|
|
rvdp n00b
Joined: 27 May 2002 Posts: 11
|
Posted: Wed Jul 03, 2002 8:50 am Post subject: |
|
|
On Tue, Jul 02, 2002 at 01:08:40AM -0700, Patrick wrote:
>
> But ideally we would use techniques that all the
> other daemons use. _exit() for child process exit
> code or something, avoid atexit calls all together.
Thanks, this will be fixed (for real) in the next beta for 3.1.35.
-- Dave
(Dave Hinds)
Till then the patch is:
1225,1229d1224
< static void dummy(void)
< {
<
< }
<
1481c1476
< atexit(&dummy);
---
> atexit(NULL);
Ive tested this patch for reboot with -f. Works nicely.
Patrick. _________________ Your pc will not crash today |
|
Back to top |
|
|
chadh Moderator
Joined: 21 Apr 2002 Posts: 137 Location: Atlanta, GA
|
Posted: Wed Jul 03, 2002 6:02 pm Post subject: |
|
|
I'll add that patch and release it as 3.1.34-r3 and 3.1.34-r4 (orinoco 0.12) _________________ Chad |
|
Back to top |
|
|
|