lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Mon, 22 Feb 2010 20:27:34 +0100
From:	Marcel Holtmann <marcel@...tmann.org>
To:	"Luis R. Rodriguez" <mcgrof@...il.com>
Cc:	David Woodhouse <dwmw2@...radead.org>,
	Johannes Berg <johannes@...solutions.net>,
	linux-kernel@...r.kernel.org,
	linux-wireless <linux-wireless@...r.kernel.org>,
	Vipin Mehta <Vipin.Mehta@...eros.com>
Subject: Re: Firmware versioning best practices II

Hi Luis,

> >> > > > That doesn't make much sense anyway. If the firmware filename is
> >> > > > foo-$APIVER-$CODEVER every code change would need a corresponding
> >> > > driver
> >> > > > change. If it is just foo-$APIVER then the $CODEVER can be embedded
> >> > > in
> >> > > > the firmware file and printed so you know which code you're using,
> >> > > but
> >> > > > if it doesn't influence the API I don't see why it should be part of
> >> > > the
> >> > > > filename?
> >> > >
> >> > > The idea is that just like with shared libraries, you have a symlink
> >> > > from the 'soname' foo-3.fw to the actual file foo-3-1.4.1.fw.
> >> >
> >> > Ah ok. I indeed do that manually with iwlwifi firmware :)
> >> >
> >> > > For shared libraries, it's easy to create those symlinks automatically
> >> > > using ldconfig. For firmware that doesn't really work though -- since
> >> > > the soname isn't encoded in the file like it is in ELF libraries.
> >> >
> >> > Right. Though I guess we could come up with a unified firmware wrapper
> >> > format that the firmware loader can unwrap.
> >>
> >> I suppose we could, but this seems like overkill to me.
> >
> > I have to agree. This looks like total overkill to me.
> >
> > Just use the $APIVER in the firmware filename.
> 
> OK -- so what goes into linux-firmware is just the latest
> 
> foo-$(API)
> 
> > And if someone wants to
> > keep track of more details then they can manually symlink them.
> 
> Well do we want the older foo-$(API)-$(VAR) files in linux-firmware
> too for those companies/developers wishing to do this?

personally I would say no, but for some projects it might make sense. I
would just leave this up to the project/driver itself.

> What about deprecating APIs of the firmware based on kernel releases.
> I see it reasonable to deprecate a firmware API completely for a
> future kernel release provided you maintain all features and
> functionality in par. Does that sound reasonable?

Realistically you have to deprecate firmware versions at some point
anyway. So yes, that makes sense. However I would leave this again up to
the driver itself and how far the maintainers wanna go in supported
older API versions.

> > Unless we have full control over the source code of every firmware used
> > in the kernel, why bother. It is up to the companies providing them
> > anyway to make sure everything works as expected and the community can't
> > fix it.
> 
> Well that's exactly it -- we do have access to the code for ar9170 for
> example, so these details will become more relevant in the future.

That is one of the few exception. However even for ar9170 there has to
be release engineering team that creates the binary and uploads it into
linux-firmware.git since I don't see the majority of people compiling it
from source. And at that point the release engineering needs to test the
API compatibility.

Regards

Marcel


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ