I think it's better to commit it now and have it fixed in situ.
It's new functionality so committing it with bugs will not break anyone.
it will however get more work done on it and more testing.
On Thu, 28 Feb 2002, Kenneth D. Merry wrote:
> On Thu, Feb 28, 2002 at 10:49:18 -0800, Julian Elischer wrote:
> > This is definitly something that is needed..
> > The question is whether the CAM and ATAPI authors feel it is
> > right. We are guided by them (even though we desperatly need this).
> > Personally even if not perfect.. it's better than nothing and we should
> > probably commit something like it. or based on it..
> > (having looked at it I think it seems fine.)
> I think it's a good idea as well.
> > So here's my vote for a quick commit.
> No. See below. There are still problems with it.
> It probably also needs to be reviewed before it goes in.
Well you are one of the main CAM peopel.. we are relying on you and
> But yes, it is a good idea.
> > On Thu, 28 Feb 2002, Thomas Quinot wrote:
> > > An updated version of the ATAPI/CAM patches is available from
> > > http://www.cuivre.fr.eu.org/~thomas/atapicam/
> > > This version contains no functional changes, but synchronize with
> > > recent modifications to the generic ATAPI code.
> > >
> > > As always, I would be interested in any feedback. Specifically, there
> > > is one known pending issue with this code: on *some* machines,
> > > patched kernels hang at boot time, immediately after registering
> > > the new CAM sim. If it hangs on your machine and you can provide
> > > a backtrace of the point where the freeze occurs, it would be most
> > > helpful.
> > >
> > > Enjoy,
> > > Thomas.
> > >
> > > --
> > > [EMAIL PROTECTED]
> > >
> > > To Unsubscribe: send mail to [EMAIL PROTECTED]
> > > with "unsubscribe freebsd-current" in the body of the message
> > >
> > To Unsubscribe: send mail to [EMAIL PROTECTED]
> > with "unsubscribe freebsd-scsi" in the body of the message
> Kenneth Merry
> [EMAIL PROTECTED]
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message