From owner-freebsd-current@FreeBSD.ORG Wed Nov 5 10:03:19 2003 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id D13C216A4CE; Wed, 5 Nov 2003 10:03:19 -0800 (PST) Received: from mailout.informatik.tu-muenchen.de (mailout.informatik.tu-muenchen.de [131.159.0.5]) by mx1.FreeBSD.org (Postfix) with ESMTP id 87FBE43FEC; Wed, 5 Nov 2003 10:03:18 -0800 (PST) (envelope-from barner@in.tum.de) Received: by zi025.glhnet.mhn.de (Postfix, from userid 1000) id 078526D; Wed, 5 Nov 2003 19:03:16 +0100 (CET) Date: Wed, 5 Nov 2003 19:03:16 +0100 From: Simon Barner To: Max Khon Message-ID: <20031105180316.GA333@zi025.glhnet.mhn.de> References: <200311040514.hA45E3Hk065630@WBIw009.westbend.net> <20031105035310.GB94240@iclub.nsu.ru> <20031105045035.GB3240@zi025.glhnet.mhn.de> <20031105095142.GA6690@iclub.nsu.ru> <20031105131002.GC411@zi025.glhnet.mhn.de> <20031105144729.GA14918@iclub.nsu.ru> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="82I3+IH0IqGh5yIs" Content-Disposition: inline In-Reply-To: <20031105144729.GA14918@iclub.nsu.ru> User-Agent: Mutt/1.5.4i X-Virus-Scanned: by amavisd-new at informatik.tu-muenchen.de cc: "Scot W. Hetzel" cc: FreeBSD-Current@freebsd.org cc: gerald@freebsd.org Subject: Re: PRs for dagrab and cdparanoia reworked (was: Re: Fix for WINE on -CURRENT) X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 05 Nov 2003 18:03:19 -0000 --82I3+IH0IqGh5yIs Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable > > I am progress of doing the same for dagrab (expect a follow-up to PR > > 57227 soon). >=20 > 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@ (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 --82I3+IH0IqGh5yIs Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.3 (FreeBSD) iD8DBQE/qTtkCkn+/eutqCoRAtu1AJ9Mhz+qLi7dYmxitYSxv/H9ZzPtoACbBzSd QmK5F2bHRfYfqOnHZcsWwzs= =h7qu -----END PGP SIGNATURE----- --82I3+IH0IqGh5yIs--