From owner-freebsd-firewire@FreeBSD.ORG Wed Jan 16 23:51:49 2008 Return-Path: Delivered-To: freebsd-firewire@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 0E13D16A421; Wed, 16 Jan 2008 23:51:49 +0000 (UTC) (envelope-from sbruno@miralink.com) Received: from plato.miralink.com (mail.miralink.com [70.103.185.20]) by mx1.freebsd.org (Postfix) with ESMTP id D14FE13C447; Wed, 16 Jan 2008 23:51:48 +0000 (UTC) (envelope-from sbruno@miralink.com) Received: from localhost (localhost.localdomain [127.0.0.1]) by plato.miralink.com (Postfix) with ESMTP id 6EE562C00BE; Wed, 16 Jan 2008 15:51:48 -0800 (PST) Received: from plato.miralink.com ([127.0.0.1]) by localhost (plato.miralink.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 18164-03; Wed, 16 Jan 2008 15:51:45 -0800 (PST) Received: from iago.office.miralink.com (iago.office.miralink.com [10.0.0.40]) by plato.miralink.com (Postfix) with ESMTP id 398632C00BD; Wed, 16 Jan 2008 15:51:45 -0800 (PST) Message-ID: <478E9890.20700@miralink.com> Date: Wed, 16 Jan 2008 15:51:44 -0800 From: Sean Bruno User-Agent: Thunderbird 2.0.0.9 (X11/20071115) MIME-Version: 1.0 To: Hidetoshi Shimokawa References: <476610E5.2060108@miralink.com> <626eb4530712162258s4dfe1448o1102f20a623d3f95@mail.gmail.com> <476696C4.60408@miralink.com> <626eb4530712182320q237c344crd309893a82fe8ef8@mail.gmail.com> <476B13F4.5050409@miralink.com> <626eb4530712201727n3fc0d33aq6e6b44603b5d77f2@mail.gmail.com> <478E2B53.7070107@miralink.com> <626eb4530801161446g7467c9cew59bd65cea68b8ed6@mail.gmail.com> In-Reply-To: <626eb4530801161446g7467c9cew59bd65cea68b8ed6@mail.gmail.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-DSPAM-Result: Innocent X-DSPAM-Processed: Wed Jan 16 15:51:46 2008 X-DSPAM-Confidence: 0.9997 X-DSPAM-Probability: 0.0000 X-DSPAM-Signature: 478e9892236791804284693 X-DSPAM-Factors: 27, X-Virus-Scanned: amavisd-new at X-Spam-Status: No, score=-4.359 tagged_above=-10 required=6.6 autolearn=ham tests=[ALL_TRUSTED=-1.8, AWL=0.140, BAYES_00=-2.599, DSPAM_HAM=-0.1] X-Spam-Score: -4.359 X-Spam-Level: Cc: freebsd-firewire@freebsd.org Subject: Re: sbp_targ memory leak X-BeenThere: freebsd-firewire@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Firewire support in FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 16 Jan 2008 23:51:49 -0000 Hidetoshi Shimokawa wrote: > Thanks for digging out the problem. > I did not suppose multiple CTIO's for an ATIO so that your guess > should be right. > > I'm sorry that I will be busy until mid of Feb. > I'd like to merge your fixes to -current in a few month. > > I appreciate if you would do the following: > - test on -current and make a patch for -current. > - split the patch for each fix > (a version control system or ports/devel/quilt may be useful) > - non-reversed patch > > Thanks, > > On Jan 17, 2008 1:05 AM, Sean Bruno wrote: > >> Hidetoshi Shimokawa wrote: >> >>> Thanks for the test. >>> I'll look into the page_table problem this weekend. >>> Sorry for late response. >>> >>> On 12/21/07, Sean Bruno wrote: >>> >>> >>>> Hidetoshi Shimokawa wrote: >>>> >>>> >>>>> I think you are right and page table is not freed when CAM_SEND_STATUS >>>>> is not set. >>>>> Maybe we should always free page tables if refcont == 0 rather than >>>>> free in sbp_targ_send_status(). >>>>> >>>>> You patch is not just adding debug printfs, right? >>>>> What is the mtx locks for? >>>>> >>>>> On 12/18/07, Sean Bruno wrote: >>>>> >>>>> >>>>> >>>>>> Hidetoshi Shimokawa wrote: >>>>>> >>>>>> >>>>>> >>>>>>> Thanks for the tracking of the problem. >>>>>>> Could you resend the patch in unified or context diff? >>>>>>> >>>>>>> Thanks, >>>>>>> >>>>>>> On 12/17/07, Sean Bruno wrote: >>>>>>> >>>>>>> >>>>>>> >>>>>>> >>>>>>>> In trying to understand and make sbp_targ functional, I've noted that >>>>>>>> the code seems to lose track of how many page tables it allocates for >>>>>>>> any give orbi. I had to add a lot of debugging code around the >>>>>>>> malloc/free's to find out what was going on, and I'm not sure what the >>>>>>>> code is supposed to do in this case. >>>>>>>> >>>>>>>> Please review the patch diff at --> http://consultcsg.com/RELENG_6.diff >>>>>>>> >>>>>>>> And the log at -->http://consultcsg.com/malloc_failure.txt >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> >>>>>> Diff updated at http://consultcsg.com/RELENG_6.diff >>>>>> >>>>>> Sean >>>>>> >>>>>> >>>>>> >>>>>> >>>> I moved the free around as you suggested and the memory leak does indeed >>>> go away and there are no further crashes. >>>> Here is my current diff --> http://consultcsg.com/RELENG_6.diff . >>>> >>>> It does look like the data is not being written or read to the backend >>>> correctly however. I.e. the page_table is not being >>>> setup correctly when more than one read or write is required to service >>>> an ORB. Any ideas on how to look into that? >>>> >>>> Sean >>>> >>>> >>>> >>>> >>> >>> >> I seem to have been able to resolve the memory leak, multiple CTIO's and >> some various lockups with the patch in this PR --> >> http://www.freebsd.org/cgi/query-pr.cgi?pr=119575 >> >> This is against RELENG_6 and should be applied. I noted 3 more issues >> that I'd like to resolve in the ticket. >> >> What do you think? >> >> Sean >> >> > > > > Excellent. I'll get on it. Probably, it will be around 4 patches? Sean