From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oded Gabbay Subject: Re: AMD GPU new API for new module Date: Sun, 12 Oct 2014 11:13:39 +0300 Message-ID: <543A3833.1060801@amd.com> References: <20141008160026.GA7643@gmail.com> <54363116.2050900@amd.com> <20141009080256.GB4576@gmail.com> <54366026.9020408@amd.com> <5436D50D.6070502@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from na01-bn1-obe.outbound.protection.outlook.com (mail-bn1bon0117.outbound.protection.outlook.com [157.56.111.117]) by gabe.freedesktop.org (Postfix) with ESMTP id 2DC796E189 for ; Sun, 12 Oct 2014 01:13:58 -0700 (PDT) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Daniel Vetter Cc: dri-devel List-Id: dri-devel@lists.freedesktop.org On 11/10/14 21:30, Daniel Vetter wrote: > On Thu, Oct 9, 2014 at 8:33 PM, Oded Gabbay wrote: >> Thanks for the input. >> I do _not_ intend to fork IOCTLs for new H/W generations, if possible. >> i.e, our driver now supports 2 h/w generations with the exact same set >> of IOCTLs and I don't see how that would change in the future.. >> >> What I'm more worried about is supporting different sets of UMD, which >> will require different IOCTLs for the same operation, e.g. CreateQueue >> for HSA runtime and OpenCL runtime. >> >> However, due to a very limited amount of UMDs, the "regular" way of >> adding IOCTLs may be sufficient. >> >> Bottom line, need to think more about it :) > > Hm, generally the ioctls should be modelled on the hw for a generic > umd. Agreed. Of course that's a bit hard in practice since predicting the > unkown is difficult ;-). But on intel hw we have about 5+ different > umd stacks if you count them all, and they all seem to be more-or-less > happy with the same ioctl interface. The problems start when you start migrating UMD from one kernel driver to another, which already has other UMDs. Then, you may need to create new IOCTLs. Like I've said it does require a > bit a mindset change though since clean-slate designs should only be > done when there's overwhelming reasons that the old interfaces just > don't cut it any more. Otoh you also need to make sure that all the > different umd teams talk to each another since ime they also err on > the other side and each come up with their own special hack to enable > a given new feature. Sounds familiar ;) > -Daniel >