From: Vu P. <hu...@ya...> - 2007-01-23 19:16:45
|
Sorry to attach/send the old log. I'll send full log next time. I think I found the problem which maybe only happen with my setup. I'm using libata and ata_piix drivers with SATA disk as my disk dev_handler back-end. In function scst_cmd_done() line 1125 cmd->bufflen = req->sr_bufflen; //?? somehow the req->sr_bufflen is zero out by local back-end scsi mid-layer or libata/ata_piix driver. Therefore, the cmd->bufflen is zero and subsequently the scst_cur_cmd_mem won't be subtracted by cmd->bufflen in scst_finish_cmd() The scst_cur_cmd_mem will keep increasing/accumulating up to scst_cur_max_cmd_mem within couple hundred read/write commands. Here is my question: 1. why do we have to retrieve cmd->bufflen from req->sr_bufflen? I dont' think that we need to do that since command has its state and controlled by scst only. cmd->bufflen will stay the same unless scst change it. I see a //?? in that code line. So you must double about it before. 2. Does anyone see the same problem with different local back-end scsi mid-layer and low level scsi driver? By removing that line, I don't see the problem since thanks, -vu --- Vladislav Bolkhovitin <vs...@vl...> wrote: > Vu Pham wrote: > > Hi Vlad, > > > > > > > >>Hi, > >> > >>I don't think this crash is related to SCST, it's > >>rather looks to be > >>ralated to "updated kernel 2.6.17". What updates > are > >>there? > > > > > > There is no update - what I meant is: > > instead of running RHAS4u2 distribution default > kernel > > (2.6.9-22.ELsmp), I run vanilla kernel 2.6.17 > > > > > >>And from the > >>traces it doesn't look like "original 0.9.5 + your > >>previous patch from > >>yesterday" was running, so recheck it. > >> > > > > > > OK. I download latest 0.9.5.1 and run it as is. > Here > > is part of the /var/log/messages showing > allocation > > failure. > > Well, you sent exactly the same file as yesterday. > Is it correct? Also, > apparently, those logs are not from 0.9.5.1. Next > time please send full > unedited kernel log from the very beginning. > > > thanks for your help, > > -vu > > > > > >>Vlad > >> > >>Vu Pham wrote: > >> > >>>Hi Vlad, > >>> I'm running RHAS4u2 + updated kernel 2.6.17 + > >>>original 0.9.5 + your previous patch from > >> > >>yesterday. > >> > >>>Here is the snipped /var/log/messages showing > >>>allocation failure and crash > >>> > >>>thanks, > >>>-vu > >>> > >>>--- Vladislav Bolkhovitin <vs...@vl...> wrote: > >>> > >>> > >>> > >>>>OK, it seems in Fedora kernel definition of > >>>>GFP_ATOMIC was changed in a > >>>>way, which is incompatible with what SCST > expects > >>> > >>>>from it. > >>> > >>>>Could you try with the attached patch, please? > >> > >>It's > >> > >>>>against clear 0.9.5. > >>>> > >>>>Thanks, > >>>>Vlad > >>>> > >> > >> > >> > > > ------------------------------------------------------------------------- > > > >>Take Surveys. Earn Cash. Influence the Future of > IT > >>Join SourceForge.net's Techsay panel and you'll > get > >>the chance to share your > >>opinions on IT & business topics through brief > >>surveys - and earn cash > >> > > > > > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > > > >>_______________________________________________ > >>Scst-devel mailing list > >>Scs...@li... > >> > > > > > https://lists.sourceforge.net/lists/listinfo/scst-devel > > > >> > >>------------------------------------------------------------------------ > >> > >>------------------------------------------------------------------------- > >>Take Surveys. Earn Cash. Influence the Future of > IT > >>Join SourceForge.net's Techsay panel and you'll > get the chance to share your > >>opinions on IT & business topics through brief > surveys - and earn cash > >>http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > >> > >> > >>------------------------------------------------------------------------ > >> > >>_______________________________________________ > >>Scst-devel mailing list > >>Scs...@li... > >>https://lists.sourceforge.net/lists/listinfo/scst-devel > > > ------------------------------------------------------------------------- > Take Surveys. Earn Cash. Influence the Future of IT > Join SourceForge.net's Techsay panel and you'll get > the chance to share your > opinions on IT & business topics through brief > surveys - and earn cash > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > _______________________________________________ > Scst-devel mailing list > Scs...@li... > https://lists.sourceforge.net/lists/listinfo/scst-devel > |