| 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/xorgComments
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(-) >
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 > >
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(-)