Re: [PATCH] amr(4) testers needed...

On Friday, 14 July 2006 at  8:51:24 -0600, Scott Long wrote:
> Nikolay Pavlov wrote:
> 
> >On Thursday, 13 July 2006 at 12:51:20 -0700, Doug Ambrisko wrote:
> >
> >>Scott Long writes:
> >>| Joao Barros wrote:
> >>| > On 7/11/06, John Baldwin <john_at_baldwin.cx> wrote:
> >>| > 
> >>| >> I have a patch for amr(4) that is a forward-port of a Scott Long 
> >>patch | >> for 4.x
> >>| >> that fixes some severe data corruption with amr(4) + PAE on 4.x with 
> >>> | >> 4GB of
> >>| >> RAM.  I need the patch tested on current though so I can get it into 
> >>| >> HEAD and
> >>| >> eventually into 4.x.  The patch for head is at
> >>| >> http://www.freebsd.org/~jhb/patches/amr_head.patch  It shouldn't 
> >>break
> >>| >> anything and should basically be a nop.  I think the patch will 
> >>apply | >> to 6.x
> >>| >> (and possibly 5.x) as well.  Thanks!
> >>| >>
> >>| > 
> >>| > I can test it on i386 without PAE. Is that test enough?
> >>| 
> >>| Note that this problem is only present when you're using a management
> >>| app at the same time as heavy disk activity is going on. If you're
> >>| not using a management app (and few people are, the lack of apps is why
> >>| I didn't catch this in the first place), then you're completely safe.
> >>| John's email might be taken as a little alarmist in this respect.
> >>
> >>Hmm, I wonder why we don't see it or maybe we are :-(  We don't usually
> >>hit the disk hard but have a management app. running.  I've seen
> >>some strangeness.
> >
> >
> >I am using this utility every hour in cron, now i have 4 processes in D
> >state 
> >
> >24429  ??  D      0:00.01 /usr/local/sbin/megarc -ldInfo -LAll -a0
> >35394  ??  D      0:00.02 /usr/local/sbin/megarc -ldInfo -LAll -a0
> >37976  ??  D      0:00.02 /usr/local/sbin/megarc -ldInfo -LAll -a0
> >40526  ??  D      0:00.02 /usr/local/sbin/megarc -ldInfo -LAll -a0
> >
> >I understand Scott position, but worried that megarc not killing by
> >SIGKILL signal, it's not normal. I want to try preceding version of amr
> >driver. Also i have 5.5 box as load balance pair for this one and never
> >see such hangs on it.
> >
> >
> 
> Instead of rolling the driver back, please add the patch that John
> provided.  It is safe and correct.

Scott this patch works just fine for me. I am using latest RELENG_6 now
and my problem with megarc have gone :) 
I don't use PAE, but server looks very stable. 
Here is some httpd statistic:

Server uptime: 2 days 15 hours 50 minutes 41 seconds
Total accesses: 322289 - Total Traffic: 982.9 GB
CPU Usage: u6615.66 s125410 cu0 cs0 - 57.4% CPU load
1.4 requests/sec - 4.4 MB/second - 3.1 MB/request
228 requests currently being processed, 159 idle servers

Thanks for you time.

> 
> Scott
> 

-- 
========================================================================= 
= Best regards, Nikolay Pavlov. <<<------------------------------------ = 
=========================================================================