From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ming Lei Subject: Re: 2.6.30-rc2-git2: Reported regressions from 2.6.29 Date: Sat, 9 May 2009 11:28:19 +0800 Message-ID: <20090509112819.759594d5@linux-lm> References: <200904202208.23899.laurent.pinchart@skynet.be> <200904220121.12002.laurent.pinchart@skynet.be> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:date:from:to:cc:subject :message-id:in-reply-to:references:x-mailer:mime-version :content-type:content-transfer-encoding; bh=ZoZRuO/ekGKnIVkInIXg20w3Q+HkQcSmuxeEYYqUTE0=; b=KEoEGAX7pJV8tM8bcytXgMC6breFNPEH9iZ3xsfmr5gIm5TP9Y0/vEs3RyzVnTAFSd wRxEyC/D7XTdukrSm9D/gZL7OHytCSzszdg+m9NW+1M4CUUsqa7Hp+9n8lDIMw6UGjZX coSEiLlbFQ0QJmfK5bSaCMkr8LMgZLZ0cZWME= In-Reply-To: <200904220121.12002.laurent.pinchart@skynet.be> Sender: linux-kernel-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" To: Laurent Pinchart Cc: Linus Torvalds , "Rafael J. Wysocki" , Linux Kernel Mailing List , Adrian Bunk , Andrew Morton , Natalie Protasevich , Kernel Testers List , Network Development , Linux ACPI , Linux PM List , Linux SCSI List , video4linux-list@redhat.com, mchehab@infradead.org On Wed, 22 Apr 2009 01:21:10 +0200 Laurent Pinchart wrote: > Hi, > > On Tuesday 21 April 2009 03:47:34 Ming Lei wrote: > > 2009/4/21 Laurent Pinchart : > > > On Saturday 18 April 2009 06:51:11 leiming wrote: > > >> From a3b3d72cdd57a0699fb643b41b78eb7beb211ff5 Mon Sep 17 > > >> 00:00:00 2001 From: Ming Lei > > >> Date: Wed, 15 Apr 2009 22:32:51 +0800 > > >> Subject: [PATCH] V4L/DVB:usbvideo:fix uvc resume failed(v2) > > >> > > >> Now urb buffers is not freed before suspend, so > > >> uvc_alloc_urb_buffers should return packet counts allocated > > >> originally during uvc resume , instead of zero. > > >> > > >> This version uses round down to return packet counts on Linus's > > >> suggestions, or else may lead to buffer destructed if packet size > > >> is changed before calling uvc_alloc_urb_buffers() in this kind of > > >> case. > > > > > > The comment is misleading. If the packet size changes we need to > > > reallocate the buffers anyway. Have you checked if the packet > > > size (which depends on the endpoint being selected) can be > > > changed between suspend and resume, either by the uvcvideo driver > > > (I don't think it can) or the USB core ? > > > > The packet size does not change between suspend and resume. I mean > > uvc_alloc_urb_buffers() still can be used in other cases if buffers > > was not freed and is reuesed in future. It seems there is no such > > cases in uvcvideo now, but uvc_alloc_urb_buffers() really __can__ > > work in such case, isn't it? > > > > IMHO It is only used to allocate or reserve UVC_URBS usb buffers, > > which size is video->urb_size, and npackets can be shortened or > > enlarged if psize is changed, after all. > > You're right. Patch applied, thanks. Rc5 has been released today, why isn't this patch accepted by upstream now? It is really a bug fix. Thanks. > > Best regards, > > Laurent Pinchart > -- Lei Ming