This is the mail archive of the cygwin-developers@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: time for 1.5.11?


On Mon, 5 Jul 2004, Corinna Vinschen wrote:

> On Jun 10 15:23, Igor Pechtchanski wrote:
> > On Thu, 10 Jun 2004, Pierre A. Humblet wrote:
> >
> > > Igor Pechtchanski wrote:
> > > >
> > > > Ok, it turns out that to fix this properly, we'll need a much larger patch
> > > > than the one line I was looking at...  Here are some thoughts.
> > > >
> > > > First off, my guess above was wrong.  The IO handle has nothing to do with
> > > > this -- fhandler_dev_dsp doesn't use it.  However, the intuition (I still
> > > > think) was correct: some fields, namely the audio_in_ and audio_out_
> > > > pointers, weren't replicated on dup(), so the new (duped) object ended up
> > > > with NULL fields, and couldn't write to them.
> > > >
> > > > The simplest solution to the above is to simply copy the pointers.  This
> > > > will work for the test program I provided earlier, but won't for another
> > > > legitimate test program (attached) that closes the original handle before
> > > > using the duped one (I think this behavior is allowed).
> > > >
> > > > The "proper" way to fix this, IMO, would be to allow sharing the audio_in_
> > > > and audio_out_ objects, and have some sort of reference count in them,
> > > > instead of deleting them outright in close().
> > >
> > > FWIW, this reminds me of the archetype concept that cgf has built into cygwin.
> > > It's currently used for ttys.
> > >
> > > Pierre
> >
> > Thanks, Pierre, I've looked at them.  They seem much more complex than
> > what's needed here, which is simply a way of sharing objects via
> > pointers...  Archetypes also seem to go out of their way to avoid an extra
> > level of indirection, but since we have it here for free anyway, reference
> > counting looks to be easier.
> > 	Igor
>
> Any patch in sight?
>
> Corinna

Corinna,

There *is* a patch that I'd like Gerd's opinion on, but I have a simpler
patch in the queue before that, and it's currently getting approved
(signed off) by my management.  As soon as that clears (and the fax is
sent), I'll start the same on this one.  Hopefully, it'll happen some time
next week.

However, don't let it hold up the release, as not too many people seem to
be affected by the dev/audio dup() problem.
	Igor
-- 
				http://cs.nyu.edu/~pechtcha/
      |\      _,,,---,,_		pechtcha@cs.nyu.edu
ZZZzz /,`.-'`'    -.  ;-;;,_		igor@watson.ibm.com
     |,4-  ) )-,_. ,\ (  `'-'		Igor Pechtchanski, Ph.D.
    '---''(_/--'  `-'\_) fL	a.k.a JaguaR-R-R-r-r-r-.-.-.  Meow!

"I have since come to realize that being between your mentor and his route
to the bathroom is a major career booster."  -- Patrick Naughton


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]