[Bug 1025720] Re: Package cedarview-drm
** Package changed: ubuntu = cedarview-drm-drivers (Ubuntu) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/cedarview-drm-drivers/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1025720] Re: Package cedarview-drm
Hi there, It seems that the cedarview packages are only available for precise, but not for quantal. Is there any reason for that? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1025720] Re: Package cedarview-drm
yes, they only work with the 3.2 kernel as that is what Intel tested the proprietary drivers with. I don't know how much effort it would be to get them working with 3.5 or newer. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1025720] Re: Package cedarview-drm
Hi I have switched from the ppa packages [1] to the repository version. jockey-text -l xorg:cedarview_gfx - Intel Cedarview graphics driver (Proprietario, Abilitato, In uso) kmod:cedarview_gfx - drm driver for the Intel GMA500 (Proprietario, Abilitato, In uso) For this graphic card: Intel Corporation Atom Processor D2xxx/N2xxx Integrated Graphics Controller (rev 09) it work: 1) with no pae 2) no 3d effect 3) aixgl off Is what you expected? You can confirm that 3D is not supported for this graphic card at this moment on 3.2.0 kernel? [1] https://launchpad.net/~sarvatt/+archive/cedarview/ -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1025720] Re: Package cedarview-drm
I've had verbal confirmation from James Leddy that he's tested with the full set of packages in precise-proposed and the install test worked through jockey; the packages have been copied to precise-updates, so considering this verified and resolved. ** Changed in: ubuntu Status: In Progress = Fix Released ** Tags added: verification-done -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1025720] Re: Package cedarview-drm
We installed the cedarview-drm from the repos on our laptop and the computer hangs during boot, is this being fixed? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1025720] Re: Package cedarview-drm
In July Intel released graphics drivers for Cedar Trail. There is a file available on the Intel web site with the file name cdv-gfx- drivers-1.0.1_bee.tar.bz2 and a date of July 6, 2012. It can be found by searching the Intel Download Center (http://downloadcenter.intel.com/Default.aspx?lang=eng) for the filename or the string Linux* PowerVR Graphics/Media Drivers. The download page links to the file, release notes and a link, Enabling hardware accelerated playback that, takes one to a page containing links to two pdf documents titled Enabling Hardware Accelerated Playback for IntelĀ® AtomTM Processor N2000/D2000 Series, one for Ubuntu and one for Fedora. Is this significant? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1025720] Re: Package cedarview-drm
Alan, this is the very same binary packaged, with a diffrenent versioning due to historical reasons (Intel only recently started publicly versioning the blobs AFAIK) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1025720] Re: Package cedarview-drm
** Branch linked: lp:ubuntu/precise-proposed/cedarview-drm-drivers -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1025720] Re: Package cedarview-drm
I've just re-uploaded the source with your changes and a new upstream release (which contains a minor fix). See the attached file for the complete diff. ** Patch added: sru2.patch https://bugs.launchpad.net/ubuntu/+bug/1025720/+attachment/3244853/+files/sru2.patch ** Changed in: ubuntu Status: New = In Progress -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
Re: [Bug 1025720] Re: Package cedarview-drm
On 27/07/12 23:21, Steve Langasek wrote: Hi Alberto, Reviewing the package in the new queue, I spotted a few issues. The first few of these need to be fixed before I will accept the package into precise. Can you please take care of these and reupload? debian/grub.d/11_custom_cmdline: this is NIHing the grub-gfxpayload blacklist handling already implemented in grub via /usr/share/grub- gfxpayload-lists/blacklist. Instead of shipping a file under /etc/grub.d which hard-codes disabling of vt handoff even when the hardware is not present, please add a blacklist file to this directory like the ones in the grub-gfxpayload-lists package and call 'update- grub-gfxpayload' (instead of update-grub') from your package's postinst and postrm (on remove/purge only, and guarded with || true). Creating the blacklist should be straightforward, you're already processing a list of PCI IDs in debian/rules. You should add a dependency on grub-gfxpayload-lists for this. The main reason why I didn't use this was that it gives us a black screen until we get a 1 or 2 seconds plymouth screen and then we get to the lightdm screen. At least this is what happens on our machines. I'll accept your changes just in case other machines with supported chipsets react in a different way. debian/lightdm.conf: why is this necessary? including this file in the package means that merely installing the driver package causes unity3d to stop working out of the box, even if this video hardware is not present. (The same reason not to hard-disable vt.handoff.) Does the Unity auto-detection of 3D support not work correctly on this hardware for some reason? If it's *really* confirmed to be needed, then ok; but we should definitely use the autodetection if possible. The driver's OpenGL implementation is, well, partial ;). This means that Unity will start (as the driver passes the Unity's tests) but you won't see anything useful on your screen. - debian/dirs is probably entirely pointless. You should only need this file if you are shipping empty directories in the package. Good point - debian/copyright references http://dep.debian.net/deps/dep5/, which is an obsolete draft. This should ideally be updated to conform to copyright-format 1.0 (http://www.debian.org/doc/packaging-manuals /copyright-format/1.0/). Ok, I'm attaching the new diff with your changes (thanks for all your work, BTW) and mine. ** Patch added: sru-changes1.patch https://bugs.launchpad.net/bugs/1025720/+attachment/3243782/+files/sru-changes1.patch -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
Re: [Bug 1025720] Re: Package cedarview-drm
On Tue, Jul 31, 2012 at 02:53:49PM -, Alberto Milone wrote: debian/grub.d/11_custom_cmdline: this is NIHing the grub-gfxpayload blacklist handling already implemented in grub via /usr/share/grub- gfxpayload-lists/blacklist. Instead of shipping a file under /etc/grub.d which hard-codes disabling of vt handoff even when the hardware is not present, please add a blacklist file to this directory like the ones in the grub-gfxpayload-lists package and call 'update- grub-gfxpayload' (instead of update-grub') from your package's postinst and postrm (on remove/purge only, and guarded with || true). Creating the blacklist should be straightforward, you're already processing a list of PCI IDs in debian/rules. You should add a dependency on grub-gfxpayload-lists for this. The main reason why I didn't use this was that it gives us a black screen until we get a 1 or 2 seconds plymouth screen and then we get to the lightdm screen. At least this is what happens on our machines. Ok. When you do it the other way, manually disabling vt.handoff but leaving gfxmode=keep, what do you see instead / what do you expect to see? I guess that you might see a *purple* screen for a few seconds, followed by a flicker and plymouth, followed by lightdm. But if this difference between purple and black is important, please file a bug against the grub2 package to discuss. This combination of gfxmode=keep + no vt_handoff is not tested in Ubuntu, and I don't think it should be introduced in a hardware-specific way. debian/lightdm.conf: why is this necessary? including this file in the package means that merely installing the driver package causes unity3d to stop working out of the box, even if this video hardware is not present. (The same reason not to hard-disable vt.handoff.) Does the Unity auto-detection of 3D support not work correctly on this hardware for some reason? If it's *really* confirmed to be needed, then ok; but we should definitely use the autodetection if possible. The driver's OpenGL implementation is, well, partial ;). This means that Unity will start (as the driver passes the Unity's tests) but you won't see anything useful on your screen. Right, understood... :) - debian/copyright references http://dep.debian.net/deps/dep5/, which is an obsolete draft. This should ideally be updated to conform to copyright-format 1.0 (http://www.debian.org/doc/packaging-manuals /copyright-format/1.0/). Ok, I'm attaching the new diff with your changes (thanks for all your work, BTW) and mine. Looks good to me. Please reupload. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1025720] Re: Package cedarview-drm
Hi Alberto, Reviewing the package in the new queue, I spotted a few issues. The first few of these need to be fixed before I will accept the package into precise. Can you please take care of these and reupload? Depends: [...] findutils (= 4), findutils is an Essential package, and version 4 is older than hardy. This dependency should be dropped. debian/grub.d/11_custom_cmdline: this is NIHing the grub-gfxpayload blacklist handling already implemented in grub via /usr/share/grub- gfxpayload-lists/blacklist. Instead of shipping a file under /etc/grub.d which hard-codes disabling of vt handoff even when the hardware is not present, please add a blacklist file to this directory like the ones in the grub-gfxpayload-lists package and call 'update- grub-gfxpayload' (instead of update-grub') from your package's postinst and postrm (on remove/purge only, and guarded with || true). Creating the blacklist should be straightforward, you're already processing a list of PCI IDs in debian/rules. You should add a dependency on grub-gfxpayload-lists for this. debian/lightdm.conf: why is this necessary? including this file in the package means that merely installing the driver package causes unity3d to stop working out of the box, even if this video hardware is not present. (The same reason not to hard-disable vt.handoff.) Does the Unity auto-detection of 3D support not work correctly on this hardware for some reason? If it's *really* confirmed to be needed, then ok; but we should definitely use the autodetection if possible. lintian reports the following issues with the maintainer scripts: W: cedarview-drm: possible-bashism-in-maintainer-script postinst:94 '' W: cedarview-drm: command-with-path-in-maintainer-script postrm:39 /usr/sbin/update-initramfs Looking more closely, I see that this section of the postinst is entirely redundant with what dh_dkms is supposed to automatically add. But dh_dkms is currently a complete no-op, because it requires the conf file to be present as debian/dkms, not debian/dkms.conf. So at a minimum, the bashisms in the postinst need to be fixed - but it would be better to remove this section of the postinst, rename debian/dkms.conf.in to debian/dkms.conf, and leverage dh_dkms that you're already build-depending on and calling. This also removes the need for an auto-generated debian/prerm. Below are some further suggestions on improving the readability / maintainability / correctness of the package. I don't consider these blockers for accepting the package into precise, but I would appreciate seeing them fixed in a subsequent upload to quantal. %: dh --with dkms $@ Note the ordering of options to dh: in recent versions, the target $@ must come as the first argument. %: dh $@ --with dkms override_dh_auto_build: clean: regen-from-templates dh_quilt_unpatch rm -f $(CURDIR)/debian/dkms.conf dh clean I think this should be written: override_dh_auto_build: override_dh_auto_clean: regen-from-templates - This is already a 3.0 (quilt) format source package, so any quilt handling here is redundant. (the build-dependency should also be dropped.) - You don't need to rm -f debian/dkms.conf from debian rules, you can just list this file name in debian/clean. - Much of the override_dh_auto_install target could be moved into a debian/install file; no real reason to have the files installed twice (once by dh_auto_install and again by dh_install). - debian/dirs is probably entirely pointless. You should only need this file if you are shipping empty directories in the package. - debian/copyright references http://dep.debian.net/deps/dep5/, which is an obsolete draft. This should ideally be updated to conform to copyright-format 1.0 (http://www.debian.org/doc/packaging-manuals /copyright-format/1.0/). I'm attaching a diff that attempts to address most of these issues. Can you look at whether this looks sane to you, and if so, can you please incorporate the changes into your package and reupload? Rejecting the package for now due to the first set of issues. These would be acceptable to fix post-upload if this were being uploaded to the devel series first, but since this is being pushed to precise and each subsequent upload requires a separate SRU, these need to be addressed the first time through. ** Patch added: cedarview-drm-fixes.diff https://bugs.launchpad.net/ubuntu/+bug/1025720/+attachment/3239085/+files/cedarview-drm-fixes.diff -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1025720] Re: Package cedarview-drm
** Tags added: patch -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 1025720] Re: Package cedarview-drm
** Changed in: ubuntu Assignee: (unassigned) = Alberto Milone (albertomilone) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1025720 Title: Package cedarview-drm To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+bug/1025720/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs