Patchwork [0/3] Add Xorg driver ABI dependencies

login
register
mail settings
Submitter Ross Burton
Date Oct. 22, 2012, 9:37 a.m.
Message ID <cover.1350898411.git.ross.burton@intel.com>
Download mbox
Permalink /patch/38391/
State New
Headers show

Pull-request

git://git.yoctoproject.org/poky-contrib ross/xorg

Comments

Ross Burton - Oct. 22, 2012, 9:37 a.m.
Hi,

The xserver driver ABIs frequently change and as such are versioned.  It's
entirely possible to build an image with an Xserver with ABI 13 but drivers
using ABI 11, and in this situation the server won't load the modules.  These
ABI versions have no relationship to the package version so the existing
dependencies can't be made stricter.

The solution Debian (and probably others) take is to add versioned Provides to
the server, and dependencies on those provides in the drivers.  Then ABI version
mismatches become package dependencies errors and are caught at image
construction time, not boot time.

Ross

The following changes since commit caba9cbfce09f19eb27f4c6615c0c5c48e1a2952:

  kernel.bbclass: add kernel-modules to PACKAGES (2012-10-19 23:06:26 +0100)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib ross/xorg

for you to fetch changes up to 4bb20610fa08a2daf71f59b42929181500345e75:

  insane: add a check for Xorg driver ABI dependencies (2012-10-22 09:26:20 +0100)

----------------------------------------------------------------
Ross Burton (3):
      xserver-xorg: add runtime provides for the driver ABI version
      xorg-driver: add xserver driver ABI dependencies
      insane: add a check for Xorg driver ABI dependencies

 meta/classes/insane.bbclass                        |   20 ++++++++++++++-
 .../xorg-driver/xorg-driver-common.inc             |   17 +++++++++++-
 .../xorg-driver/xorg-driver-input.inc              |    4 +++
 .../xorg-driver/xorg-driver-video.inc              |    3 +++
 .../recipes-graphics/xorg-xserver/xserver-xorg.inc |   27 +++++++++++++++++++-
 5 files changed, 68 insertions(+), 3 deletions(-)

Ross Burton (3):
  xserver-xorg: add runtime provides for the driver ABI version
  xorg-driver: add xserver driver ABI dependencies
  insane: add a check for Xorg driver ABI dependencies

 meta/classes/insane.bbclass                        |   20 ++++++++++++++-
 .../xorg-driver/xorg-driver-common.inc             |   17 +++++++++++-
 .../xorg-driver/xorg-driver-input.inc              |    4 +++
 .../xorg-driver/xorg-driver-video.inc              |    3 +++
 .../recipes-graphics/xorg-xserver/xserver-xorg.inc |   27 +++++++++++++++++++-
 5 files changed, 68 insertions(+), 3 deletions(-)
Saul Wold - Oct. 23, 2012, 6:53 p.m.
On 10/22/2012 02:37 AM, Ross Burton wrote:
> Hi,
>
> The xserver driver ABIs frequently change and as such are versioned.  It's
> entirely possible to build an image with an Xserver with ABI 13 but drivers
> using ABI 11, and in this situation the server won't load the modules.  These
> ABI versions have no relationship to the package version so the existing
> dependencies can't be made stricter.
>
> The solution Debian (and probably others) take is to add versioned Provides to
> the server, and dependencies on those provides in the drivers.  Then ABI version
> mismatches become package dependencies errors and are caught at image
> construction time, not boot time.
>
> Ross
>
> The following changes since commit caba9cbfce09f19eb27f4c6615c0c5c48e1a2952:
>
>    kernel.bbclass: add kernel-modules to PACKAGES (2012-10-19 23:06:26 +0100)
>
> are available in the git repository at:
>
>    git://git.yoctoproject.org/poky-contrib ross/xorg
>
> for you to fetch changes up to 4bb20610fa08a2daf71f59b42929181500345e75:
>
>    insane: add a check for Xorg driver ABI dependencies (2012-10-22 09:26:20 +0100)
>
> ----------------------------------------------------------------
> Ross Burton (3):
>        xserver-xorg: add runtime provides for the driver ABI version
Merged into OE-Core
>        xorg-driver: add xserver driver ABI dependencies
Not merged yet.

>        insane: add a check for Xorg driver ABI dependencies
>
Merged into OE-Core

Thanks
	Sau!
>   meta/classes/insane.bbclass                        |   20 ++++++++++++++-
>   .../xorg-driver/xorg-driver-common.inc             |   17 +++++++++++-
>   .../xorg-driver/xorg-driver-input.inc              |    4 +++
>   .../xorg-driver/xorg-driver-video.inc              |    3 +++
>   .../recipes-graphics/xorg-xserver/xserver-xorg.inc |   27 +++++++++++++++++++-
>   5 files changed, 68 insertions(+), 3 deletions(-)
>
> Ross Burton (3):
>    xserver-xorg: add runtime provides for the driver ABI version
>    xorg-driver: add xserver driver ABI dependencies
>    insane: add a check for Xorg driver ABI dependencies
>
>   meta/classes/insane.bbclass                        |   20 ++++++++++++++-
>   .../xorg-driver/xorg-driver-common.inc             |   17 +++++++++++-
>   .../xorg-driver/xorg-driver-input.inc              |    4 +++
>   .../xorg-driver/xorg-driver-video.inc              |    3 +++
>   .../recipes-graphics/xorg-xserver/xserver-xorg.inc |   27 +++++++++++++++++++-
>   5 files changed, 68 insertions(+), 3 deletions(-)
>
Saul Wold - Oct. 24, 2012, 3:10 p.m.
On 10/23/2012 11:53 AM, Saul Wold wrote:
> On 10/22/2012 02:37 AM, Ross Burton wrote:
>> Hi,
>>
>> The xserver driver ABIs frequently change and as such are versioned.
>> It's
>> entirely possible to build an image with an Xserver with ABI 13 but
>> drivers
>> using ABI 11, and in this situation the server won't load the
>> modules.  These
>> ABI versions have no relationship to the package version so the existing
>> dependencies can't be made stricter.
>>
>> The solution Debian (and probably others) take is to add versioned
>> Provides to
>> the server, and dependencies on those provides in the drivers.  Then
>> ABI version
>> mismatches become package dependencies errors and are caught at image
>> construction time, not boot time.
>>
>> Ross
>>
>> The following changes since commit
>> caba9cbfce09f19eb27f4c6615c0c5c48e1a2952:
>>
>>    kernel.bbclass: add kernel-modules to PACKAGES (2012-10-19 23:06:26
>> +0100)
>>
>> are available in the git repository at:
>>
>>    git://git.yoctoproject.org/poky-contrib ross/xorg
>>
>> for you to fetch changes up to 4bb20610fa08a2daf71f59b42929181500345e75:
>>
>>    insane: add a check for Xorg driver ABI dependencies (2012-10-22
>> 09:26:20 +0100)
>>
>> ----------------------------------------------------------------
>> Ross Burton (3):
>>        xserver-xorg: add runtime provides for the driver ABI version
> Merged into OE-Core
>>        xorg-driver: add xserver driver ABI dependencies
> Not merged yet.
>
Merged into OE-Core

Thanks
	Sau!
>>        insane: add a check for Xorg driver ABI dependencies
>>
> Merged into OE-Core
>
> Thanks
>      Sau!
>>   meta/classes/insane.bbclass                        |   20
>> ++++++++++++++-
>>   .../xorg-driver/xorg-driver-common.inc             |   17 +++++++++++-
>>   .../xorg-driver/xorg-driver-input.inc              |    4 +++
>>   .../xorg-driver/xorg-driver-video.inc              |    3 +++
>>   .../recipes-graphics/xorg-xserver/xserver-xorg.inc |   27
>> +++++++++++++++++++-
>>   5 files changed, 68 insertions(+), 3 deletions(-)
>>
>> Ross Burton (3):
>>    xserver-xorg: add runtime provides for the driver ABI version
>>    xorg-driver: add xserver driver ABI dependencies
>>    insane: add a check for Xorg driver ABI dependencies
>>
>>   meta/classes/insane.bbclass                        |   20
>> ++++++++++++++-
>>   .../xorg-driver/xorg-driver-common.inc             |   17 +++++++++++-
>>   .../xorg-driver/xorg-driver-input.inc              |    4 +++
>>   .../xorg-driver/xorg-driver-video.inc              |    3 +++
>>   .../recipes-graphics/xorg-xserver/xserver-xorg.inc |   27
>> +++++++++++++++++++-
>>   5 files changed, 68 insertions(+), 3 deletions(-)
>>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>