unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Jelle Licht <jlicht@fsfe.org>
To: Tobias Geerinckx-Rice <me@tobias.gr>, 38403@debbugs.gnu.org
Subject: [bug#38403] [PATCH] gnu: Add intel-vaapi-driver-g45-h264.
Date: Fri, 29 Nov 2019 12:00:43 +0100	[thread overview]
Message-ID: <87lfrzueuc.fsf@jlicht.xyz> (raw)
In-Reply-To: <875zj59mes.fsf@nckx>

Hey Tobias,

Tobias Geerinckx-Rice via Guix-patches via <guix-patches@gnu.org>
writes:

> Jelle,
>
> Jelle Licht 写道:
>> I am not sure if this package offers any practical benefit or 
>> not, so I
>> defer to someone who actually knows what they are talking about 
>> to make
>> a judgment call ;-).
>
> I don't have a G45, but if this driver verifiably works for you 
> (and the regular version does not) I think it's good to to include 
> it.  Also because I expect this card to be overrepresented amongst 
> Guix users: there aren't that many common Librebootable machines.
>
> It needs its own clear synopsis, description, and home page 
> though.  Is there a home page?  I'm not good at navigating 
> BitBucket.

I am in contact with the upstream author (of this tarball): They do not
have access to a G45-based chipset, so we are currently looking into
options for the future. There is currently no home page.

The actual bitbucket repo has been made public only yesterday, so it was
not that weird that you could not find it :-).
>
>> This variant of intel-vaapi-driver is a backport of the ancient 
>> g45-h264
>> branch that used to be maintained by the intel-vaapi-driver 
>> team. As far
>> as I know, the reason they don't maintain this branch anymore is 
>> that
>> hardware accelerated h264 decoding on the G45 chipset was much 
>> too slow
>> for 1080p video, as by default it only had 32MB of VRAM to work 
>> with.
>
> These devices don't have their own RAM chips, so ‘by default’ here 
> means the BIOS menu setting, right?  If so, it should be mentioned 
> in the description for lack of upstream documentation.
This is correct. I think it makes sense to point people to the right
documentation for at least libreboot and coreboot, and refer them to
their BIOS supplier for other instructions.

OTOH, I do not feel that the package description is the right place to
do this. Where would such instructions normally exist on GNU/Linux
machines?

> I'm guessing that your card identifies as an Intel® GMA X4500MHD. 
> Is that correct?  We should include the marketing name in the 
> synopsis & description too, even if it might not map 1:1 to the 
> chipset.
Yes on all fronts.

- Jelle

  reply	other threads:[~2019-11-29 11:14 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-27 16:59 [bug#38403] [PATCH] gnu: Add intel-vaapi-driver-g45-h264 Jelle Licht
2019-11-27 18:58 ` Tobias Geerinckx-Rice via Guix-patches via
2019-11-29 11:00   ` Jelle Licht [this message]
2020-04-15 22:32     ` Tobias Geerinckx-Rice via Guix-patches via
2020-04-13 17:52 ` Vincent Legoll
2020-04-13 20:01   ` Jelle Licht
2020-04-15 22:06     ` Vincent Legoll
2020-09-12 11:20       ` Jelle Licht
2022-01-31 17:51 ` iung--- via Guix-patches via

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87lfrzueuc.fsf@jlicht.xyz \
    --to=jlicht@fsfe.org \
    --cc=38403@debbugs.gnu.org \
    --cc=me@tobias.gr \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).