Re: PRs for dagrab and cdparanoia reworked (was: Re: Fix for WINE on -CURRENT)

From: Simon Barner <barner_at_in.tum.de>
Date: Wed, 5 Nov 2003 19:03:16 +0100
> > I am progress of doing the same for dagrab (expect a follow-up to PR
> > 57227 soon).
> 
> There is no need for extra-patches.
> #ifdef CDIOCREADAUDIO seems to be sufficient enough.

The version in GNATs is obsoleted, but my update did not make it there,
although it appeared in freebsd-ports-bugs_at_ (I used the follow-up
email-link in the GNATs web interface).

I will prepare and test a version that used pread() like yours this night.
It will be here then:

http://home.leo.org/~barner/freebsd/dagrab-pread.patch

> Is using CDRIOCSETBLOCKSIZE necessary as well?
> If yes it shouold be done in enable_cdda callback I think.

I don't now. I don't know how to interpret Soren's comments in

  http://freebsd.rambler.ru/bsdmail/freebsd-current_2003/msg15989.html

regarding our situation:

  > The right way of doing audio graps has been to set the wanted
  > blocksize with CDRIOCSETBLOCKSIZE (not needed if all tracks on
  > the CD is of the same size), then just read from the device.
  > Ioctl's was newer meant to be used to read/write data, its also
  > faster to use the R/W path...

Can we assue that all tracks on an audio CD are of the same (block)
size? What about those mixed-mode CDs (sorry, I don't know the
specifications).

> Moreover, is there a reason we should use CDIOCREADAUDIO on RELENG_4?
> Please test attached patch. works for me on RELENG_4 and should work
> on -CURRENT as well.

Yep, it works fine for me on 4.9-STABLE and 5.1-CURRENT (and complies
with Soren's posting).

I think the best solution would be to patch all applications to use this
approach. The only question is whether we need CDRIOCSETBLOCKSIZE.

Simon

Received on Wed Nov 05 2003 - 09:03:19 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:37:27 UTC