Subject: Re: CVS commit: [netbsd-3] src/sys/arch/xen/xen
To: None <source-changes@netbsd.org>
From: Jed Davis <jdev@panix.com>
List: source-changes
Date: 08/17/2005 18:07:32
Thor Lancelot Simon <tls@netbsd.org> writes:

> On Wed, Aug 17, 2005 at 10:01:37AM +0900, YAMAMOTO Takashi wrote:
>> > Module Name:	src
>> > Committed By:	tron
>> > Date:		Mon Aug 15 14:57:19 UTC 2005
>> > 
>> > Modified Files:
>> > 	src/sys/arch/xen/xen [netbsd-3]: xbdback.c
>> > 
>> > Log Message:
>> > Pull up revision 1.12 (requested by tls in ticket #656):
>> > Improvements to xbdback (the domain 0 driver that provides "xbd" virtual
>> > disks to other domains) from Jed Davis, <jld@panix.com>:
>> 
>> IMO, changes known to have bugs should be pulled up with their fixes.
>
> I'm sorry, I probably missed something while I was out of town for
> several weeks and trying to work on the road.  What was the bug, and
> which revision to which file fixed it?

xen/xbdback.c, r1.13: fixes an error-case control-flow mistake on my part
   (see http://mid.gmane.org/%3clcsu0iql9xk.fsf%40panix5.panix.com%3e )
           and r1.14: fixes a race condition I didn't notice

i386/xen_shm_machdep.c, r1.11: fixes assorted merge-derived errors
   (see http://mid.gmane.org/%3clcsoe8yl954.fsf%40panix5.panix.com%3e )


-- 
(let ((C call-with-current-continuation)) (apply (lambda (x y) (x y)) (map
((lambda (r) ((C C) (lambda (s) (r (lambda l (apply (s s) l))))))  (lambda
(f) (lambda (l) (if (null? l) C (lambda (k) (display (car l)) ((f (cdr l))
(C k)))))))    '((#\J #\d #\D #\v #\s) (#\e #\space #\a #\i #\newline)))))