public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Stefan Richter <stefanr@s5r6.in-berlin.de>
Cc: Jan Hoogenraad <jan-conceptronic@hoogenraad.net>,
	Hans Verkuil <hverkuil@xs4all.nl>,
	linux-media@vger.kernel.org
Subject: Re: firedtv and removal of old IEEE1394 stack
Date: Wed, 09 Feb 2011 14:05:04 -0200	[thread overview]
Message-ID: <4D52BB30.6080900@redhat.com> (raw)
In-Reply-To: <20110209152459.36abeda5@stein>

Em 09-02-2011 12:24, Stefan Richter escreveu:
> On Feb 09 Stefan Richter wrote:
>>> https://bugs.launchpad.net/ubuntu/+source/linux-kernel-headers/+bug/134222
> 
> Correction:
> 
> Bug 134222 has *nothing* to do with the removal of the older ieee1394
> stack.  

Agreed.

> The bug is about
> 
>   1. a defect during assembling the linux-kernel-headers package.

Yes.

>      The drivers/ieee1394/* files do not belong into such a package.
>      They are driver source files, not exported kernel headers.
> 
>      Don't export kernel source files as linux-headers if they are not
>      meant to be exported.

It would be fine if they match the ABI symbols found on the vanilla kernel
for the same version. If they don't match, the media backport system could
have a fix, as there are several cases where the backport system detects
for some specific ABI changes, and apply some solution that will make the
driver compile and work with that ABI version.
 
>   2. the dvb backports relying on this broken package.  Tough luck.
> 
>      You want to build a kernelspace driver whose sources include other
>      kernel sources?  Well, include these kernel sources, not some
>      arbitrary userland source files.

I partially agree

The media backport tree is meant only to help people to test media drivers. 
It is not meant to be used on production, as nobody is actually doing tests 
to check if the backports are fine for some specific distro.

So, it is an "use with your own risk" approach. If it doesn't work... well,
this can happen. People are free to fix and send us patches for it.

The real solution is to do a real backport for some specific distro. The
media_build might help to point where the ABI differences are, but some
additional changes may be required to be sure that the module will work
fine.

On the other hand, adding the Firewire stack at the media building tree
would probably add more pain, especially for the ones using other firewire
hardware and/or other distros.

In any case, I don't think that Stefan or any Firewire upstream developer
should do anything to solve it. It is a problem that needs to be addressed
by Ubuntu people, and/or by the developers that want to test the Firwire driver
with the Ubuntu-shipped kernel.

Cheers,
Mauro.

  reply	other threads:[~2011-02-09 16:05 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-03 16:06 firedtv and removal of old IEEE1394 stack Hans Verkuil
2011-02-05 14:21 ` Stefan Richter
2011-02-05 14:32   ` Stefan Richter
2011-02-09  6:40     ` Jan Hoogenraad
2011-02-09 13:22       ` Stefan Richter
2011-02-09 14:24         ` Stefan Richter
2011-02-09 16:05           ` Mauro Carvalho Chehab [this message]
2011-02-09 15:51         ` Mauro Carvalho Chehab
2011-02-05 14:36   ` Hans Verkuil

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

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

  git send-email \
    --in-reply-to=4D52BB30.6080900@redhat.com \
    --to=mchehab@redhat.com \
    --cc=hverkuil@xs4all.nl \
    --cc=jan-conceptronic@hoogenraad.net \
    --cc=linux-media@vger.kernel.org \
    --cc=stefanr@s5r6.in-berlin.de \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox