linux-kernel - Re: [linux-usb-devel] [PATCH] Properly unregister reboot notifier in case of failure in ehci hcd

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] [day] [month] [year] [list]

Date:	Mon, 31 Jul 2006 14:30:53 -0700
From:	David Brownell <david-b@...bell.net>
To:	Alan Stern <stern@...land.harvard.edu>
Cc:	Aleksey Gorelov <dared1st@...oo.com>,
	linux-usb-devel@...ts.sourceforge.net,
	Andrew Morton <akpm@...l.org>, gregkh@...e.de,
	linux-kernel@...r.kernel.org
Subject: Re: [linux-usb-devel] [PATCH] Properly unregister reboot notifier in case of failure in ehci hcd

On Monday 31 July 2006 12:46 pm, Alan Stern wrote:
> On Mon, 31 Jul 2006, Aleksey Gorelov wrote:
> 
> > > What code duplication?  Doing it the way I suggested doesn't require 
> > > adding any new code at all.  You, on the other hand, added several 
> > > routines for bus glue that does virtually nothing.
> > 
> >   But you can not use exactly same shutdown function with both pci and platform glue. You need to
> > convert pci/platform device to hcd anyway, right ? So this will add 2 doing 'virtually nothing'
> > routines anyway (unless you just want to duplicate the code of shutdown routine for for platform
> > glue). For ohci, you would need to do the same, hence 2 more routines, 4 total. With bus glue, I
> > added just 2. Am I missing something here ?
> 
> Okay, now I understand your point.  Yes, it makes sense to do it your way.

I confess that I had thought about doing it purely at the "bus glue" level
rather than as a new HCD method, but having an HCD method for this does make
sense in terms of simpler code in HCDs that cope with more than one kind of
bus glue.  (And I'd forgotten that OHCI needs a third type, because of that
SA1111 "minibus".)

So as for Aleksey's new patch:

Signed-off-by: David Brownell <dbrownell@...rs.sourceforge.net>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/