linux-kernel - Re: [PATCH] serial 8250: tighten test for using backup timer

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: <20080826105845.6ddf1107.akpm@linux-foundation.org>
Date:	Tue, 26 Aug 2008 10:58:45 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	David Brownell <david-b@...bell.net>
Cc:	linux-kernel@...r.kernel.org, alex.williamson@...com,
	Will Newton <will.newton@...il.com>
Subject: Re: [PATCH] serial 8250: tighten test for using backup timer

On Tue, 26 Aug 2008 10:45:34 -0700
David Brownell <david-b@...bell.net> wrote:

> > > Also, how serious is the problem which is being fixed here?  It
> > > _sounds_ like it's of the "fatal for people who have that hardware"
> > > variety, in which case we should get this into 2.6.27 and probably
> > > 2.6.26.x.  Not sure about 2.5.26.x though - the patch doesn't apply
> > > there, but I didn't check whether this is due to functional changes.
> >
> > For users of this version of this particular UART IP it is fatal. From
> > looking at the git history it looks like the original patch went into
> > 2.6.26 so it might also affect that kernel.
> 
> Second that:  serial-8250-tighten-test-for-using-backup-timer.patch
> (from MMOTM) in mainline sooner-not-later seems right.

ok, if you think so...

I still have an unanswered question out there about the init_timer()
handling in that area of the driver but afaict this patch didn't make any
of it worse than it already is.

> My own exposure to this is that the UART on DaVinci hardware, which
> TI allegedly derived from its original 16550 logic, has periodically
> gone from working to unusable with the mainline 8250.c ... and back
> and forth a bunch.  Currently it's "unusable", a regression from some
> previous versions.  With this patch from Alex, it's usable.
> 
> Of course there are a bunch of arch/arm/mach-davinci patches needed
> to make that platform more functional in mainline.  I expect they'll
> probably merge in the 2.6.28-rc0 window.  Meanwhile, running out of
> ramdisk with a serial console should at least work right!  :)
> 

--
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/