linux-kernel - Re: [PATCH v6 17/22] usb: dwc2: host: Manage frame nums better in scheduler

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20160209095341.GA12324@herrerog>
Date:	Tue, 9 Feb 2016 10:53:41 +0100
From:	"Herrero, Gregory" <gregory.herrero@...el.com>
To:	Doug Anderson <dianders@...omium.org>
Cc:	John Youn <John.Youn@...opsys.com>, Felipe Balbi <balbi@...com>,
	Kever Yang <kever.yang@...k-chips.com>,
	吴良峰 <william.wu@...k-chips.com>,
	Tao Huang <huangtao@...k-chips.com>,
	Heiko Stübner <heiko@...ech.de>,
	Stefan Wahren <stefan.wahren@...e.com>,
	"open list:ARM/Rockchip SoC..." <linux-rockchip@...ts.infradead.org>,
	linux-rpi-kernel@...ts.infradead.org,
	Julius Werner <jwerner@...omium.org>,
	"Kaukab, Yousaf" <yousaf.kaukab@...el.com>,
	Dinh Nguyen <dinguyen@...nsource.altera.com>,
	Alan Stern <stern@...land.harvard.edu>,
	Ming Lei <ming.lei@...onical.com>,
	John Youn <johnyoun@...opsys.com>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v6 17/22] usb: dwc2: host: Manage frame nums better in
 scheduler

Hi Doug,

> Hi,
> 
> On Thu, Jan 28, 2016 at 6:20 PM, Douglas Anderson <dianders@...omium.org> wrote:
> >  static void dwc2_qh_init(struct dwc2_hsotg *hsotg, struct dwc2_qh *qh,
> >                          struct dwc2_hcd_urb *urb)
> >  {
> > @@ -569,11 +655,6 @@ static void dwc2_qh_init(struct dwc2_hsotg *hsotg, struct dwc2_qh *qh,
> >                               qh->ep_type == USB_ENDPOINT_XFER_ISOC,
> >                               bytecount));
> >
> > -               /* Ensure frame_number corresponds to the reality */
> > -               hsotg->frame_number = dwc2_hcd_get_frame_number(hsotg);
> 
> In reviewing patches I realized that this is actually a revert of
> commit dd81dd7c8178 ("usb: dwc2: host: use correct frame number during
> qh init").  IMHO that patch was wrong: hsotg->frame_number is supposed
> to be the frame number as of the last start of frame.  If we need to
> know a more recent frame number then we should query it ourselves.
> 
> Presumably the reason for the original patch was to try to fix some of
> the same problems I've addressed in my series, so I'd presume that
> this doesn't add any new regressions.  I haven't heard much from
> Gregory Herrero about my series, but it would be nice to confirm that
> this virtual revert wasn't causing problems.
> 

This patch ("usb: dwc2: host: use correct frame number during qh init")
is no more needed with your patchset.
Note that your patchset is also reverting commit 08c4ffc:
("usb: dwc2: host: reset frame number after suspend")
but it is no more needed as well with your patchset.

I tried suspend/resume with different devices and didn't face the issue
my previous commit was fixing.

Regards,
Gregory