From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wayne Gong Subject: Re: GPLPV Drivers 0.9.11-pre4 hang during boot - WindowsXP Pro SP2 Date: Fri, 04 Jul 2008 23:15:54 +0800 Message-ID: <486E3EAA.2020303@oracle.com> References: <485A8681.10503@darkskiez.co.uk> <485AD22D.7070303@darkskiez.co.uk> <485F726C.2000908@darkskiez.co.uk> <4861D361.9070806@sigala.it> <486C7997.80503@sigala.it> <486CDFE4.6040006@sigala.it> <486DD394.4060403@sigala.it> <486DDF74.1010909@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: James Harper Cc: Sandro Sigala , xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org > That patch appears to be to your wdf version of the code... can you give > me a summary of what the change you made is? > > At a guess, the difference to the original code is that you are doing > two separate notify's, and a KeMemoryBarrier after each. Can you tell my > why the barrier is required? > > I just noticed that the for() loop will never actually do anything as > req->nr_segments is initialised to zero... that's the same in my current > code too! I wonder how long that has been a problem for? I'll change my > code to initialise nr_segments to the maximum. > > James > > Yes, as you mentioned, there are only two difference. When I meet the bug, I have no idea to deal with it. So I try several method and times to fix it. After I changed the code up to the patch as I send to you, it works fine. And till now I am not very such whether this bug has been fixed. So I this patch just a try and I cannot explain it exactly even today. Best regards -- Wayne