Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 22 Feb 2004 12:33:18 -0700
From:      Scott Long <scottl@freebsd.org>
To:        Mathew Kanner <mat@cnd.mcgill.ca>
Cc:        cvs-all@freebsd.org
Subject:   Re: cvs commit: src/sys/alpha/alpha mem.c promcons.csrc/sys/alpha/tlsbsrc/sys/cam/scsi scsi_ch.c scsi_pass.c scsi_pt.c         s
Message-ID:  <403903FE.3090306@freebsd.org>
In-Reply-To: <20040222185750.GE20226@cnd.mcgill.ca>
References:  <200402212110.i1LLAtEX008732@repoman.freebsd.org> <20040221154636.U52892@pooker.samsco.home> <20040222010801.GC20226@cnd.mcgill.ca> <403800F0.9080504@freebsd.org> <20040222185750.GE20226@cnd.mcgill.ca>

next in thread | previous in thread | raw e-mail | index | archive | help
Matt,

I already committed the fix to ltmdm last night.  One difference from
your patch is that the D_TTY flag was added.  For the nvidia patch, I'd
also remove the definition for CDEV_MAJOR from nv_freebsd.h.

Thanks for looking at these!

Scott

Mathew Kanner wrote:
>>There is also ltmdm and nvidia, and both are affected by this.  I can
>>probably look at both tonight unless someone beats me to it.
> 
> 
> 	Attached are patches for the ltmdm and nvidia ports.  It
> should suffice to drop them in the files subdir of the port.  I've at
> least tested the nvidia one since I have the hardware.  They should be
> also tested on a pre-cdev mega-patch kernel.
> 	I should note that when nvidia module was rejected (wrong
> version) on it panic'ed on kldunload with free wrong memtype.  I'm
> sorry for not capturing the details.
> 	It's my hope that some with porting experience can get these
> into the tree, pehaps with a friendly reminder to update you
> out-of-tree modules in UPDATING.  I'm not sure where source picks up
> the freebsd-version so you may have to re-installed the sys header
> files.
> 	--Mat
> 
> 
> 
> ------------------------------------------------------------------------
> 
> --- src/nvidia_ctl.c.old	Sun Feb 22 13:05:46 2004
> +++ src/nvidia_ctl.c	Sun Feb 22 13:07:25 2004
> @@ -43,7 +43,12 @@
>      .d_ioctl =     nvidia_ctl_ioctl,
>      .d_poll =      nvidia_ctl_poll,
>      .d_name =      "nvidiactl",
> +#if __FreeBSD_version < 502103
>      .d_maj =       CDEV_MAJOR,
>      .d_flags =     D_TRACKCLOSE
> +#else
> +    .d_version =   D_VERSION, 
> +    .d_flags =     D_NEEDGIANT
> +#endif 
>  #endif
>  };
> 
> 
> ------------------------------------------------------------------------
> 
> --- src/nvidia_dev.c.old	Wed May 28 12:51:52 2003
> +++ src/nvidia_dev.c	Sun Feb 22 13:36:25 2004
> @@ -45,7 +45,12 @@
>      .d_poll =      nvidia_dev_poll,
>      .d_mmap =      nvidia_dev_mmap,
>      .d_name =      "nvidia",
> +#if __FreeBSD_version < 502103
>      .d_maj =       CDEV_MAJOR,
>      .d_flags =     D_MEM|D_TRACKCLOSE
> +#else
> +    .d_version =   D_VERSION,
> +    .d_flags   =   D_MEM|D_TRACKCLOSE|D_NEEDGIANT
> +#endif
>  #endif
>  };
> 
> 
> ------------------------------------------------------------------------
> 
> --- sys/dev/ltmdm/ltmdmsio.c.old	Sun Feb 22 13:12:24 2004
> +++ sys/dev/ltmdm/ltmdmsio.c	Sun Feb 22 13:13:38 2004
> @@ -407,11 +407,16 @@
>      .d_ioctl = sioioctl,
>      .d_poll = ttypoll,
>      .d_name = driver_name,
> +#if __FreeBSD_version < 502103
>  #ifdef MAJOR_AUTO
>      .d_maj = MAJOR_AUTO,
>  #else
>      .d_maj = CDEV_MAJOR,
>  #endif
> +#else
> +    .d_version = D_VERSION, 
> +    .d_flags = D_NEEDGIANT,
> +#endif
>      .d_kqfilter = ttykqfilter,
>  #endif
>  };




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?403903FE.3090306>