linux-kernel - Re: [PATCHv2] OMAP: Enable Magic SysRq on serial console ttyOx

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: <20110121074530.GB12307@m-desktop>
Date:	Fri, 21 Jan 2011 13:15:30 +0530
From:	"G, Manjunath Kondaiah" <manjugk@...com>
To:	Govindraj <govindraj.ti@...il.com>
Cc:	Anand Gadiyar <gadiyar@...com>,
	Manjunath Kondaiah G <manjugk@...com>,
	Thomas Weber <weber@...science.de>, linux-omap@...r.kernel.org,
	Tony Lindgren <tony@...mide.com>,
	Govindraj Raja <govindraj.raja@...com>,
	Greg Kroah-Hartman <gregkh@...e.de>,
	Andrei Emeltchenko <andrei.emeltchenko@...ia.com>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCHv2] OMAP: Enable Magic SysRq on serial console ttyOx

On Fri, Jan 21, 2011 at 12:54:29PM +0530, Govindraj wrote:
> On Thu, Jan 20, 2011 at 5:49 PM, Anand Gadiyar <gadiyar@...com> wrote:
> >> > >>>> Magic SysRq key is not working for OMAP on new serial
> >> > >>>> console ttyOx because SUPPORT_SYSRQ is not defined
> >> > >>>> for omap-serial.
> >> > >>>>
> >> > >>>> This patch defines SUPPORT_SYSRQ in omap-serial and
> >> > >>>> enables handling of Magic SysRq character.
> >> > >>>>
> >> > >>>> Signed-off-by: Thomas Weber <weber@...science.de>
> >> > >>>
> >> > >>> Looks fine to me.
> >> > >>>
> >> > >>> Acked-by: Govindraj.R <govindraj.raja@...com>
> >> > >> I tried to use SysRq key on minicom after applying this patch, it
> > looks
> >> > >> like it is not triggering sysrq event.
> >> > >>
> >> > >> Am I missing anything?
> >> > >>
> >> > >> -Manjunath
> >> > >> --
> >> > > Hello Manjunath,
> >> > >
> >> > > Do you have CONFIG_MAGIC_SYSRQ enabled?
> >> > > Magic SysRq key in Kernel Hacking
> >> > >
> >> > > I tested it on Devkit8000 (beagle board clone).
> >> > >
> >> >
> >> > re-setting lsr_break_flag to 0 in receive chars is causing issues
> >> > in getting sysrq key break sequence on omap-serial.c
> >> >
> >> > Manju,
> >> >
> >> > can you try this change on your environment.
> >> > With below change works for me on 3430SDP/4430SDP.
> >> > key sequence I checked.
> >> > [alt + b + t]  => shows trace of tasks running.
> >> > [alt + b + b] => system reboot.
> >>
> >> With below patch, it works fine on TeraTerm. However, I am not able to
> >> perform the same on minicom.
> >>
> >> Tested-by: Manjunath G Kondaiah <manjugk@...com>
> >>
> >> -Manjunath
> >>
> >
> > Not sure what you guys are trying out, but I am able to use sysrq
> > just fine in minicom (using only Thomas' original patch) - you need
> > to send a break sequence, and the way to do this in minicom is to
> > do Ctrl-A followed by F.
> >
> > Works for me, without the "resetting lsr_break_flag to 0" part.
> >
> > @Govind,
> >
> > What are the issues you see in getting sysrq key break sequence
> > without your change? And how is your change fixing this?
> 
> Actually I was using teraterm on windows platform.
> using keyboard to send a break char [alt + b] sometimes
> i observed that first break char was getting lost and subsequent break
> chars where getting
> recognized. However using send break option from terterm menu seems to
> work most of times.
> my change can be dropped.
If it is fixing the issue of losing first break character, what is wrong in
having this patch?

Do you see any other issues because of this change?

-Manjunath

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