linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marcel Holtmann <holtmann@linux.intel.com>
To: Tomas Winkler <tomasw@gmail.com>
Cc: Christoph Hellwig <hch@infradead.org>,
	reinette chatre <reinette.chatre@linux.intel.com>,
	"John W. Linville" <linville@tuxdriver.com>,
	Johannes Berg <johannes@sipsolutions.net>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: New iwlwifi 3945 uCode available
Date: Wed, 22 Oct 2008 20:25:08 +0200	[thread overview]
Message-ID: <1224699908.9386.145.camel@californication> (raw)
In-Reply-To: <1ba2fa240810220528w6b2e4e60k7d2390328d9b5c49@mail.gmail.com>

Hi Tomas,

> >> I do not believe our request is unreasonable. If a user runs a new
> >> kernel the log will print a message that the firmware is incorrect ...
> >> all the user needs to do is go to
> >> http://intellinuxwireless.org/?n=Downloads and download the latest
> >> firmware. This does not require any subtle changes that may affect other
> >> aspects of the system ... just the firmware file that is only used by
> >> the driver.
> >
> > with the wireless that doesn't work.  Nice.  Sorry guys, but requiring
> > a new firmware version for a new driver release is completely
> > unreasonable.  You can drop support for older firmware ABIs after a few
> > years,but not immediately.
> 
> Years.. this HW will be obsolete in year or two, hopefully recycled.
> I personally don't want people using this buggy firmware for years.

seems I have to put clearly my community hat on now.

This is not true at all. I am still using a ipw2100 in one of my
machine. So the 3945 and 4965 will be in use for a long time. The
community measurement of obsolete and companies measurement are totally
different. Also don't forget these countries where laptops and
electronics are still expensive. The old Intel wireless cards will still
be used for a long time there.

> This is development branch after all, it's not even scheduled for 2.6.28.
> 
> Ucode API for 4965 was changed to -2 without much pain, So there is a
> precedence this work quite well. This time there were some hiccups in
> handling this since the API version wasn't updated it as should be.
> really our fault. But I don't think this situation is so dramatic as
> you describe it.

Actually that is your impression. I was falling over the 4965 ucode API
change twice and it took me longer than 5 minutes to even debug and
understand what went wrong here. Since there is no real warning anywhere
and the wireless device does show up. And I do know how to handle these
things.

> The objective of the objections is not that we want to immensely hurt
> users but rather the trade off between maintain even uglier code and
> users running buggy firmware versus logging into older kernel and
> downloading new firmware in 5 minutes looks to me not proportional.
> Distro users that pull updates won't even notice. I might be wrong
> though.

There will be cases where it is too much of a burden and too complicated
to keep support for both APIs, but in all other cases we should try to
have the driver work with it. I am fully behind Johannes and Christoph
here since that is what the community expects.

So if we ran into a case where we need to abandon the support for the
old API, then we first should make sure that the modules_install will
warn loudly about the missing firmware. And with MODULE_FIRMWARE and the
firmware work from David, we do have everything in place to actually do
this. So lets do it.

An additional option would be to create drivers/deprecated/ and put a
copy of the old driver there and have some Kconfig magic to allow
selecting only one of the drivers.

Regards

Marcel



  reply	other threads:[~2008-10-22 18:24 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-21 18:27 New iwlwifi 3945 uCode available reinette chatre
2008-10-21 21:25 ` Johannes Berg
2008-10-21 21:33   ` Tomas Winkler
2008-10-21 21:34     ` Johannes Berg
2008-10-21 21:38       ` John W. Linville
2008-10-21 21:53         ` Tomas Winkler
2008-10-21 22:13           ` Marcel Holtmann
2008-10-21 22:23             ` Tomas Winkler
2008-10-21 23:01               ` Marcel Holtmann
2008-10-21 23:09                 ` Luis R. Rodriguez
2008-10-21 23:11                   ` Marcel Holtmann
2008-10-21 23:27                     ` Luis R. Rodriguez
2008-10-21 23:31                       ` Marcel Holtmann
2008-10-21 23:34                         ` Luis R. Rodriguez
2008-10-21 22:28             ` reinette chatre
2008-10-21 23:06               ` Marcel Holtmann
2008-10-21 23:34                 ` Tomas Winkler
2008-10-22  1:03                   ` Marcel Holtmann
2008-10-22  7:27               ` Johannes Berg
2008-10-22  7:56                 ` Marcel Holtmann
2008-10-22  7:57                   ` Johannes Berg
2008-10-22 20:30                   ` reinette chatre
2008-10-22 22:59                     ` Marcel Holtmann
2008-10-22 23:14                       ` reinette chatre
2008-10-22 23:39                         ` John W. Linville
2008-10-23  1:12                         ` Marcel Holtmann
2008-10-23  7:22                         ` Johannes Berg
2008-10-22  9:10                 ` Tomas Winkler
2008-10-22  9:14                   ` Johannes Berg
2008-10-22  9:23                     ` Tomas Winkler
2008-10-22 10:07               ` Christoph Hellwig
2008-10-22 12:28                 ` Tomas Winkler
2008-10-22 18:25                   ` Marcel Holtmann [this message]
2008-10-24 12:45                   ` Florian Mickler
2008-10-21 21:59       ` reinette chatre
  -- strict thread matches above, loose matches on Subject: below --
2009-04-28 20:52 new " reinette chatre

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=1224699908.9386.145.camel@californication \
    --to=holtmann@linux.intel.com \
    --cc=hch@infradead.org \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=reinette.chatre@linux.intel.com \
    --cc=tomasw@gmail.com \
    /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;
as well as URLs for NNTP newsgroup(s).