From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH - alsa-lib 1/1] Change card/pid get functions to return -ENOSYS if the kernel is too old Date: Fri, 08 Apr 2016 15:10:27 +0200 Message-ID: References: <1459532030-6387-1-git-send-email-agoode@google.com> <20160401211558.GA20661@mail.zuhause> <20160407201041.GB26683@mail.zuhause> <5707AA7C.5030804@perex.cz> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.suse.de (mx2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id A8DCC264F23 for ; Fri, 8 Apr 2016 15:10:27 +0200 (CEST) In-Reply-To: <5707AA7C.5030804@perex.cz> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Jaroslav Kysela Cc: Adam Goode , alsa-devel@alsa-project.org, Martin Koegler List-Id: alsa-devel@alsa-project.org On Fri, 08 Apr 2016 14:56:28 +0200, Jaroslav Kysela wrote: > > Dne 8.4.2016 v 12:24 Takashi Iwai napsal(a): > > On Thu, 07 Apr 2016 22:10:41 +0200, > > Martin Koegler wrote: > >> > >> On Thu, Apr 07, 2016 at 03:23:01PM -0400, Adam Goode wrote: > >>> Have you heard any objections? > >>> > >>> Also, would you plan to do a 1.1.2 release soon after accepting this, to > >>> get this functionality out quickly? > >> > >> Distributions already start to pick 1.1.1 up: > >> https://build.opensuse.org/request/show/382608 > >> https://build.opensuse.org/request/show/382611 > >> > >> I would object merging that patch, if there is no immediate patched 1.1.2 release available, > >> as otherwise 1.1.1 with a different API will get used by the mass. > > > > I have no strong opinion on this, so far. Either way (a quick 1.1.2 > > fix release, or introducing a new function in 1.1.2) isn't a big > > deal. Maybe the former is easier, but the latter is safer. > > > > Jaroslav, what's your take? > > (And no, let's not use the versioned symbols again :) > > What about 1.1.1.2 ? Do you mean 1.1.1.1? It's already confusing, as you see :) > This + "pcm_plugin: fix appl pointer not correct > when mmap_commit() return error" patch ? If you don't mind a quick release, we may do it, yes, no matter which version number is. But, meanwhile, I thought of the change again, and now wonder whether it's really right to return an error *and* -1. -1 conflicts with EPERM. And I thought there is no POSIX definition of error numbers, so in theory, -ENOSYS may be -1. thanks, Takashi