Mir

"mesa" platform is no longer accurate and needs renaming

Bug #1381330 reported by Daniel van Vugt on 2014-10-15

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Mir	Fix Released	Medium	Cemil Azizoglu	Mir 0.14.0
	mir (Ubuntu)	Fix Released	Undecided	Unassigned

Bug Description

Our "mesa" code and packages mostly represents support for DRM/DRI output. And the only use case for this till now also used Mesa for the libGL* implementation.

This is no longer true however. With Nvidia and AMD soon to release DRM support in their proprietary drivers, we will be supporting them. These drivers will provide DRM display access, but as usual will probably provide their own custom libGL* implementations. So won't use Mesa.

This should work for Mir. However when a vendor provides their own libGL*, we are technically no longer using Mesa at all. We will still be using the "mesa" code path in Mir though. So that all needs renaming to something more accurate like "drm" or "dri" etc. Not "mesa".

Tags:

Related branches

lp://staging/~cemil-azizoglu/mir/split-mesa-into-common-and-kms

Merged into lp://staging/mir at revision 2615

Daniel van Vugt: Abstain on 2015-06-03

Alexandros Frantzis (community): Approve on 2015-06-02

PS Jenkins bot (community): Approve (continuous-integration) on 2015-06-01

Alan Griffiths: Approve on 2015-06-01

Chris Halse Rogers: Approve on 2015-05-31

lp://staging/~cemil-azizoglu/mir/rename-mesa-to-KMS

Merged into lp://staging/mir at revision 2624

PS Jenkins bot (community): Approve (continuous-integration) on 2015-06-04

Alan Griffiths: Approve on 2015-06-04

Daniel van Vugt: Abstain on 2015-06-03

Alexandros Frantzis (community): Needs Information on 2015-06-02

Chris Halse Rogers: Approve on 2015-05-31

lp://staging/ubuntu/wily-proposed/mir

Daniel van Vugt (vanvugt) on 2014-10-15

summary:

- "mesa" is no longer accurate and needs renaming
+ "mesa" platform is no longer accurate and needs renaming

Revision history for this message

Alexandros Frantzis (afrantzis) wrote on 2014-10-15:

> This is no longer true however. With Nvidia and AMD soon to release DRM support in their proprietary drivers,
> we will be supporting them. These drivers will provide DRM display access, but as usual will probably provide
> their own custom libGL* implementations. So won't use Mesa.

Will they provide gbm too?

> Our "mesa" code and packages mostly represents support for DRM/DRI output.

Our mesa code is a mix of DRM/KMS, EGL and gbm. The important point for this discussion is that gbm is a mesa-specific technology.

> So that all needs renaming to something more accurate like "drm" or "dri" etc. Not "mesa".

I think "drm" is not specific enough, and "dri" is an implementation detail we never interact with directly.

If other vendors implement the full DRM, EGL, gbm triad, they are essentially implementing the "mesa" interface,
so I still think "mesa" is the most appropriate name. If they diverge, e.g., don't have gbm, we would need a new platform anyway.

Alternatives we could consider are drm-mesa, so that we could then have drm-nvidia etc for diverging implementations all based on drm.

But let's wait and see what the drivers have to offer in terms of APIs before making any decisions.

Revision history for this message

Daniel van Vugt (vanvugt) wrote on 2014-10-15:

Saying that a driver vendor "implements the mesa interface" while there's almost no linkage to Mesa remaining is obviously confusing.

Also, gbm being part of Mesa is just a minor implementation detail. Even if a vendor used that implementation, most of their driver would be non-Mesa code.

So in both cases the word "mesa" needs to go away.

Revision history for this message

Cemil Azizoglu (cemil-azizoglu) wrote on 2015-06-08:

With the renaming we've done, I think for now at least we are in good shape.

Changed in mir:
status:	Triaged → Fix Committed
milestone:	none → 0.14.0

Daniel van Vugt (vanvugt) on 2015-06-09

Changed in mir:
assignee:	nobody → Cemil Azizoglu (cemil-azizoglu)

Revision history for this message

Launchpad Janitor (janitor) wrote on 2015-07-23:

This bug was fixed in the package mir - 0.14.0+15.10.20150722-0ubuntu1

---------------
mir (0.14.0+15.10.20150722-0ubuntu1) wily; urgency=medium

  [ Andreas Pokorny ]
  * Fix missing ABI renaming in Mirplatform
  * Bump Mirserver platform graphics to 3
  * Fix mirprotobuf ABI break

[ CI Train Bot ]
* New rebuild forced.

-- CI Train Bot <email address hidden> Wed, 22 Jul 2015 18:01:49 +0000

Changed in mir (Ubuntu):
status:	New → Fix Released

Daniel van Vugt (vanvugt) on 2015-07-24

Changed in mir:
status:	Fix Committed → Fix Released

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.