On Tue, Jul 6, 2021 at 5:25 PM Jason Gunthorpe <jgg@xxxxxxxx> wrote: > On Tue, Jul 06, 2021 at 04:39:19PM +0200, Daniel Vetter wrote: > > On Tue, Jul 6, 2021 at 4:23 PM Jason Gunthorpe <jgg@xxxxxxxx> wrote: > > > > > > On Tue, Jul 06, 2021 at 12:36:51PM +0200, Daniel Vetter wrote: > > > > > > > If that means AI companies don't want to open our their hw specs > > > > enough to allow that, so be it - all you get in that case is > > > > offloading the kernel side of the stack for convenience, with zero > > > > long term prospects to ever make this into a cross vendor subsystem > > > > stack that does something useful. > > > > > > I don't think this is true at all - nouveau is probably the best > > > example. > > > > > > nouveau reverse engineered a userspace stack for one of these devices. > > > > > > How much further ahead would they have been by now if they had a > > > vendor supported, fully featured, open kernel driver to build the > > > userspace upon? > > > > There is actually tons of example here, most of the arm socs have > > fully open kernel drivers, supported by the vendor (out of tree). > > I choose nouveau because of this: > > $ git ls-files drivers/gpu/drm/arm/ | xargs wc -l > 15039 total > $ git ls-files drivers/gpu/drm/nouveau/ | xargs wc -l > 204198 total drm/arm is the arm display driver, which isn't actually shipping anywhere afaik. Also it's not including the hdmi/dp output drivers, those are generally external on socs, but integrated in discrete gpu. The other thing to keep in mind is that one of these drivers supports 25 years of product generations, and the other one doesn't. So I think adding it all up it's not that much different. Last time I looked if you look at just command submission and rendering/compute, and not include display, which heavily skews the stats, it's about 10% kernel, 90% userspace driver parts. Not including anything that's shared, which is most of it (compiler frontend, intermediate optimizer, entire runtime/state tracker and all the integration and glue pieces largely). > At 13x the size of mali this is not just some easy to wire up memory > manager and command submission. And after all that typing it still > isn't very good. The fully supported AMD vendor driver is over 3 > million lines, so nouveau probably needs to grow several times. AMD is 3 million lines the size because it includes per-generation generated header files. And of course once you throw an entire vendor team at a driver all those engineers will produce something, and there's the usual that the last 10% of features produce about 90% of the complexity and code problem. E.g. the kbase driver for arm mali gpu is 20x the size of the in-tree panfrost driver - they need to keep typing to justify their continued employement, or something like that. Usually it's because they reinvent the world. > My argument is that an in-tree open kernel driver is a big help to > reverse engineering an open userspace. Having the vendors > collaboration to build that monstrous thing can only help the end goal > of an end to end open stack. Not sure where this got lost, but we're totally fine with vendors using the upstream driver together with their closed stack. And most of the drivers we do have in upstream are actually, at least in parts, supported by the vendor. E.g. if you'd have looked the drm/arm driver you picked is actually 100% written by ARM engineers. So kinda unfitting example. > For instance a vendor with an in-tree driver has a strong incentive to > sort out their FW licensing issues so it can be redistributed. Nvidia has been claiming to try and sort out the FW problem for years. They even managed to release a few things, but I think the last one is 2-3 years late now. Partially the reason is that there don't have a stable api between the firmware and driver, it's all internal from the same source tree, and they don't really want to change that. > I'm not sure about this all or nothing approach. AFAIK DRM has the > worst problems with out of tree drivers right now. Well I guess someone could stand up a drivers/totally-not-gpu and just let the flood in. Even duplicated drivers and everything included, because the vendor drivers are better. Worth a shot, we've practically started this already, I'm just not going to help with the cleanup. > > Where it would have helped is if this open driver would come with > > redistributable firmware, because that is right now the thing making > > nouveau reverse-engineering painful enough to be non-feasible. Well > > not the reverse-engineering, but the "shipping the result as a working > > driver stack". > > I don't think much of the out of tree but open drivers. The goal must > be to get vendors in tree. Agreed. We actually got them in-tree largely. Nvidia even contributes the oddball thing, and I think the tegra line is still fully supported in upstream with the upstream driver. I'm not sure the bleak picture you're drawing is reality, aside from the fact that Nvidia discrete gpu drivers being a disaster with no redistributable firmware, no open kernel driver that works, and nothing else really either. > I would applaud Habana for getting an intree driver at least, even if > the userspace is not what we'd all want to see. > > > I don't think the facts on the ground support your claim here, aside > > from the practical problem that nvidia is unwilling to even create an > > open driver to begin with. So there isn't anything to merge. > > The internet tells me there is nvgpu, it doesn't seem to have helped. Not sure which one you mean, but every once in a while they open up a few headers, or a few programming specs, or a small driver somewhere for a very specific thing, and then it dies again or gets obfuscated for the next platform, or just never updated. I've never seen anything that comes remotely to something complete, aside from tegra socs, which are fully supported in upstream afaik. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx
