Warner Losh wrote:
>
> # please excuse this message in English to the traditionally Japanese list.
> # Julians changes broke pc98 builds with his KSE commits.
>
> In message <[EMAIL PROTECTED]> Warner Losh writes:
> : Actaully, the issues are that machdep and friends weren't updated.
>
> Please review http://people.freebsd.org/~imp/pc98.diff
>
> and pay special attention to what I did to fd.c. I wasn't sure. The
> rest were mechanical no-brainers.
>
> I booted this on my PC-9821Nr, but had no console compiled with
> kernel. A kernel from before the change also has this problem, so I
> don't know what to think.
>
> Warner
All looks good.. obviously the logic of what was going on was easy
enough to pick up :-)
the only bit I didn't see so easily was:
static int
-fdformat(dev, finfo, p)
+fdformat(dev, finfo, td)
dev_t dev;
struct fd_formb *finfo;
- struct proc *p;
+ struct thread *td;
{
+ struct proc *p = td->td_proc;
fdu_t fdu;
fd_p fd;
@@ -2860,7 +2861,7 @@
/*
* keep the process from being swapped
*/
- PHOLD(p);
+ PROC_LOCK(p);
bzero((void *)bp, sizeof(*bp));
bp->bio_cmd = BIO_FORMAT;
@@ -2898,7 +2899,7 @@
/*
* allow the process to be swapped
*/
- PRELE(p);
+ PROC_UNLOCK(p);
free(bp, M_TEMP);
return rv;
}
the replacement of PHOLD with PROC_LOCK().. I'm not sure of what this actualy
means..
the rest is as you say...mechanical..
--
+------------------------------------+ ______ _ __
| __--_|\ Julian Elischer | \ U \/ / hard at work in
| / \ [EMAIL PROTECTED] +------>x USA \ a very strange
| ( OZ ) \___ ___ | country !
+- X_.---._/ presently in San Francisco \_/ \\
v
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message
