Date: Sun, 07 Dec 2003 20:45:43 +0100 From: "Poul-Henning Kamp" <phk@phk.freebsd.dk> To: Mathew Kanner <mat@cnd.mcgill.ca> Cc: freebsd-current@freebsd.org Subject: Re: major number leak with modules? Message-ID: <26738.1070826343@critter.freebsd.dk> In-Reply-To: Your message of "Sun, 07 Dec 2003 14:37:48 EST." <20031207193748.GB98718@cnd.mcgill.ca>
next in thread | previous in thread | raw e-mail | index | archive | help
In message <20031207193748.GB98718@cnd.mcgill.ca>, Mathew Kanner writes: >On Dec 07, Poul-Henning Kamp wrote: >> In message <20031207173959.GE42518@cicely12.cicely.de>, Bernd Walter writes: >> >The situation was the following during driver development. >> >It's an USB driver and kldload'ed. >> >A plugged in device got major 247 for the nodes it created. >> >On unplugging the nodes were destroyed. >> >kldunloading the driver and kldloading the next revision created >> >nodes with major 246 for new devices. >> > >> >Do we have a leak with major numbers or is the old major number free >> >after last destroy_dev and assigning algorithm just took the next. >> >> Yes, repeatedly loading/unloading will leak majors. >> >> I have some ref-counting code to solve this problem. Warners >> axe-swinging in the old ISA drivers made it easier, but some necessary >> but uncommitted patches to the sound code from cg@ are still at >> road-block. > > Hello Poul-Henning, > It's my understanding that Cameron has submitted patches to >you. If you have no objection to them, please commit them. >Otherwise, I will try to work some patches that are favorable to you. Yes, I have his patches here but have done no testing apart from compilining them, It was my impression that Cameron wanted to commit himself ? I've put the patch here, in case you havn't seen it: http://phk.freebsd.dk/patch/sound.patch -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 phk@FreeBSD.ORG | TCP/IP since RFC 956 FreeBSD committer | BSD since 4.3-tahoe Never attribute to malice what can adequately be explained by incompetence.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?26738.1070826343>