Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 141020 - mISDN patch for kernels without devfs, version bump
Summary: mISDN patch for kernels without devfs, version bump
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Stefan Schweizer (RETIRED)
URL:
Whiteboard:
Keywords: EBUILD
Depends on:
Blocks:
 
Reported: 2006-07-19 04:24 UTC by Martin von Gagern
Modified: 2006-07-25 02:36 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
nodevfs.patch (nodevfs.patch,1.16 KB, patch)
2006-07-19 04:26 UTC, Martin von Gagern
Details | Diff
misdn-20060715.ebuild (misdn-20060715.ebuild,3.44 KB, text/plain)
2006-07-19 04:30 UTC, Martin von Gagern
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Martin von Gagern 2006-07-19 04:24:56 UTC
While examining bug 139406 I tried to switch to vanilla-sources-2.6.18_rc2 and use mISDN. For this I had to patch the mISDN sources to work without devfs. While I was at it, I got the latest CVS as well. Seems to work so far, although right now I'm back to gentoo-sources-2.6.17-r2.
Comment 1 Martin von Gagern 2006-07-19 04:26:57 UTC
Created attachment 92199 [details, diff]
nodevfs.patch

This patch simply removes any devfs related function calls from drivers/isdn/hardware/mISDN/udevice.c which seems to work well enough.
Comment 2 Martin von Gagern 2006-07-19 04:30:56 UTC
Created attachment 92200 [details]
misdn-20060715.ebuild

This ebuild changes the following in relation to 20060321:
1. apply the patch from comment 1
2. add new configurable modules HFCMINI and NETJET,
   as listed in the mISDN Makefile
3. changed all driver configs from y to m as set in the mISDN Makefile
4. removed old patches that are no longer needed
5. changed udev syntax from KERNEL="obj-*" to KERNEL=="obj-*"
Comment 3 Stefan Schweizer (RETIRED) gentoo-dev 2006-07-19 04:46:59 UTC
http://genstef.homelinux.org/local/net-dialup/misdn/

I also have some ebuilds here - but they are a bit outdated and I do not add them because they stopped working for me a month ago. I will try with your ebuild later.

Thanks for the report :)
Comment 4 Martin von Gagern 2006-07-19 05:05:31 UTC
(In reply to comment #3)
> I also have some ebuilds here - but they are a bit outdated and I do not add
> them because they stopped working for me a month ago.

Could it be they stopped working after devfs has been removed in 2.6.13?
Maybe we could combine them, if there are useful improvements in yours.
For example, how about this NEWCTRL_BRANCH? What is that about? Useful?
Comment 5 Stefan Schweizer (RETIRED) gentoo-dev 2006-07-19 05:55:29 UTC
yeah, merging both ebuilds sounds good. Could you give it a try? My ebuild has the big difference that it uses the misdn-init scripts provided in the misdn cvs and thus makes gentoo compatible with the other distributions. If that works out I would like to go that route.
Comment 6 Stefan Schweizer (RETIRED) gentoo-dev 2006-07-19 12:05:09 UTC
Can you please also get this patch upstream?
http://news.gmane.org/gmane.linux.isdn.i4l.devel
Comment 7 Martin von Gagern 2006-07-19 12:24:18 UTC
(In reply to comment #6)
> Can you please also get this patch upstream?
> http://news.gmane.org/gmane.linux.isdn.i4l.devel

I thought "upstream" in this case would have been
isdn4linux@listserv.isdn4linux.de
I did send my patch there, although mailman garbled my signature and pipermail stripped my body... :-(

Anyway, I'll gladly submit this to the newsgroup as well.
Oh, after submitting it to .devel, I found my original mail got mirrored to .user as well: http://article.gmane.org/gmane.linux.isdn.i4l.user/2221
Tough luck. but I guess I won't further spam the lists by mentioning this double post...
Comment 8 Stefan Schweizer (RETIRED) gentoo-dev 2006-07-24 01:02:58 UTC
I have committed a merged version to cvs, I hope it works for you :)

I also changed the patch to use kernel version 2.6.18 and not 2.6.13

Thanks for the bug!
Comment 9 Martin von Gagern 2006-07-24 14:02:08 UTC
(In reply to comment #8)
> I have committed a merged version to cvs,

You forgot to add the nodevfs patch to the portage tree. Otherwise thanks!

> I hope it works for you :)

Compiled all right, for my 2.6.17.6 vanilla sources.
I'll see if it works after rebooting my system.

> I also changed the patch to use kernel version 2.6.18 and not 2.6.13

So the functions were kept in the kernel for backwards compatibility till 2.6.18? Looks that way, otherwise it would not have compiled for me right now.
Comment 10 Martin von Gagern 2006-07-25 02:36:35 UTC
(In reply to comment #8)
> I have committed a merged version to cvs, I hope it works for you :)

No, it does not work. If I emerge this ebuild and start capi, capiinfo returns no controllers present, and capisuite won't start. If I remove /etc/modules.d/misdn, run "modules-update force", and revert to simply loading the required modules myself (in autoload or in a small init script), things work fine.

So far I haven't figured out the difference between those two approaches, and why exactly yours fails. I guess this will need some further investigation with a lot of reboots, not a thing I'm looking forward to.