linux-kernel - Re: [PATCH v3 10/15] h8300: Build scripts

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: <1424166381.7082.27.camel@x220>
Date:	Tue, 17 Feb 2015 10:46:21 +0100
From:	Paul Bolle <pebolle@...cali.nl>
To:	Yoshinori Sato <ysato@...rs.sourceforge.jp>
Cc:	linux-kernel@...r.kernel.org, linux-arch@...r.kernel.org
Subject: Re: [PATCH v3 10/15] h8300: Build scripts

On Tue, 2015-02-17 at 16:39 +0900, Yoshinori Sato wrote:
> At Sun, 15 Feb 2015 20:23:04 +0100,
> Paul Bolle wrote:
> > On Sun, 2015-02-15 at 16:49 +0900, Yoshinori Sato wrote:
> > > diff --git a/arch/h8300/Kconfig.debug b/arch/h8300/Kconfig.debug
> > > new file mode 100644
> > > index 0000000..eb72b01
> > > --- /dev/null
> > > +++ b/arch/h8300/Kconfig.debug
> > > @@ -0,0 +1,23 @@
> > > +menu "Kernel hacking"
> > > +
> > > +source "lib/Kconfig.debug"
> > > +
> > > +config FULLDEBUG
> > > +	bool "Full Symbolic/Source Debugging support"
> > > +	help
> > > +	  Enable debugging symbols on kernel build.
> > > +
> > > +config HIGHPROFILE
> > > +	bool "Use fast second timer for profiling"
> > > +	help
> > > +	  Use a fast secondary clock to produce profiling information.
> > > +
> > > +config NO_KERNEL_MSG
> > > +	bool "Suppress Kernel BUG Messages"
> > > +	help
> > > +	  Do not output any debug BUG messages within the kernel.
> > > +
> > > +config SH_STANDARD_BIOS
> > > +       def_bool n
> > > +
> > 
> > I don't think this entry is needed.
> 
> This entry using sh-sci. 

This entry will always set SH_STANDARD_BIOS to 'n', and a line reading
    # CONFIG_SH_STANDARD_BIOS is not set

will always be included in the generated .config for h8300. But for the
three lines in drivers/tty/serial/sh-sci.c reading
    #ifdef CONFIG_SH_STANDARD_BIOS
            sh_bios_gdb_detach();
    #endif

that line has no effect. Even if there's no entry for SH_STANDARD_BIOS
the preprocessor will still remove that call of sh_bios_gdb_detach().

> > > +endmenu

Thanks,


Paul Bolle

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