Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [RFC/AIX] xm-aix4.h - Remove vfork redefine
@ 2004-08-08  4:55 Joel Brobecker
  2004-08-08 12:37 ` Mark Kettenis
  0 siblings, 1 reply; 4+ messages in thread
From: Joel Brobecker @ 2004-08-08  4:55 UTC (permalink / raw)
  To: gdb-patches; +Cc: Andrew Cagney, Peter Schauer, Kevin Buettner

config/xm-aix4.h contains the following define:

| /* There is no vfork.  */
| 
| #define vfork   fork

I think we should add a (link) test in configure for that. But the other
alternative is to do as we did for HP/UX (but the reason we did it that
way for HP/UX is because vfork is broken on this system).

-- 
Joel


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [RFC/AIX] xm-aix4.h - Remove vfork redefine
  2004-08-08  4:55 [RFC/AIX] xm-aix4.h - Remove vfork redefine Joel Brobecker
@ 2004-08-08 12:37 ` Mark Kettenis
  2004-08-08 13:37   ` Andrew Cagney
  2004-08-09  5:51   ` Joel Brobecker
  0 siblings, 2 replies; 4+ messages in thread
From: Mark Kettenis @ 2004-08-08 12:37 UTC (permalink / raw)
  To: brobecker; +Cc: gdb-patches

   Date: Sat, 7 Aug 2004 21:55:34 -0700
   From: Joel Brobecker <brobecker@gnat.com>

   config/xm-aix4.h contains the following define:

   | /* There is no vfork.  */
   | 
   | #define vfork   fork

   I think we should add a (link) test in configure for that. But the other
   alternative is to do as we did for HP/UX (but the reason we did it that
   way for HP/UX is because vfork is broken on this system).

Autoconf already handles that.  There should be a "#define vfork fork"
in your config.h.

Mark


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [RFC/AIX] xm-aix4.h - Remove vfork redefine
  2004-08-08 12:37 ` Mark Kettenis
@ 2004-08-08 13:37   ` Andrew Cagney
  2004-08-09  5:51   ` Joel Brobecker
  1 sibling, 0 replies; 4+ messages in thread
From: Andrew Cagney @ 2004-08-08 13:37 UTC (permalink / raw)
  To: Mark Kettenis; +Cc: brobecker, gdb-patches

>    Date: Sat, 7 Aug 2004 21:55:34 -0700
>    From: Joel Brobecker <brobecker@gnat.com>
> 
>    config/xm-aix4.h contains the following define:
> 
>    | /* There is no vfork.  */
>    | 
>    | #define vfork   fork
> 
>    I think we should add a (link) test in configure for that. But the other
>    alternative is to do as we did for HP/UX (but the reason we did it that
>    way for HP/UX is because vfork is broken on this system).
> 
> Autoconf already handles that.  There should be a "#define vfork fork"
> in your config.h.

In the case of HP, while autoconf indicates the presence of vfork, it 
doesn't (can't) detect the brokenness(1) that stops it working with GDB 
:-(  That may be the situtation here, but then it easily also be a 
hangover from a defunct AIX version, or even a cut/paste error :-/

Joel, I'd just try it and see.

Andrew


(1) The combination of vfork, exec and ptrace


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [RFC/AIX] xm-aix4.h - Remove vfork redefine
  2004-08-08 12:37 ` Mark Kettenis
  2004-08-08 13:37   ` Andrew Cagney
@ 2004-08-09  5:51   ` Joel Brobecker
  1 sibling, 0 replies; 4+ messages in thread
From: Joel Brobecker @ 2004-08-09  5:51 UTC (permalink / raw)
  To: Mark Kettenis; +Cc: gdb-patches

> Autoconf already handles that.  There should be a "#define vfork fork"
> in your config.h.

You are right, configure does handle this. And you know what, I see a
vfork on my AIX 4.3.2 system! So, unless vfork is broken (I'll know
very soon), it should actually be removed.

-- 
Joel


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2004-08-09  5:51 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-08-08  4:55 [RFC/AIX] xm-aix4.h - Remove vfork redefine Joel Brobecker
2004-08-08 12:37 ` Mark Kettenis
2004-08-08 13:37   ` Andrew Cagney
2004-08-09  5:51   ` Joel Brobecker

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox