| 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
| ||
|
Message-ID: <20140321132816.GW15608@titan.lakedaemon.net> Date: Fri, 21 Mar 2014 09:28:16 -0400 From: Jason Cooper <jason@...edaemon.net> To: Teodora Băluţă <teobaluta@...il.com> Cc: Dave Jones <davej@...hat.com>, linux-kernel@...r.kernel.org, "Waskiewicz Jr, Peter P" <peter.p.waskiewicz.jr@...el.com> Subject: Re: [RFC] QR encoding for Oops messages On Wed, Mar 19, 2014 at 10:38:30PM +0200, Teodora Băluţă wrote: > On Wed, Mar 19, 2014 at 10:18 PM, Dave Jones <davej@...hat.com> wrote: > > On Mon, Mar 17, 2014 at 02:59:47PM -0700, Teodora Baluta wrote: > > > This feature encodes Oops messages into a QR barcode that is scannable by > > > any device with a camera. > > > > ... > > > > > include/linux/print_oops.h | 11 + > > > include/linux/qrencode.h | 546 +++++++++++++ > > > kernel/Makefile | 1 + > > > kernel/panic.c | 5 + > > > kernel/print_oops.c | 173 +++++ > > > kernel/printk/printk.c | 9 +- > > > lib/Kconfig | 5 + > > > lib/Kconfig.debug | 11 + > > > lib/Makefile | 3 + > > > lib/qr/Makefile | 6 + > > > lib/qr/bitstream.c | 233 ++++++ > > > lib/qr/bitstream.h | 37 + > > > lib/qr/mask.c | 320 ++++++++ > > > lib/qr/mask.h | 39 + > > > lib/qr/mmask.c | 175 +++++ > > > lib/qr/mmask.h | 36 + > > > lib/qr/mqrspec.c | 259 +++++++ > > > lib/qr/mqrspec.h | 155 ++++ > > > lib/qr/qrencode.c | 871 +++++++++++++++++++++ > > > lib/qr/qrencode.h | 546 +++++++++++++ > > > lib/qr/qrinput.c | 1834 ++++++++++++++++++++++++++++++++++++++++++++ > > > lib/qr/qrinput.h | 129 ++++ > > > lib/qr/qrspec.c | 543 +++++++++++++ > > > lib/qr/qrspec.h | 178 +++++ > > > lib/qr/rscode.c | 325 ++++++++ > > > lib/qr/rscode.h | 38 + > > > lib/qr/split.c | 331 ++++++++ > > > lib/qr/split.h | 44 ++ > > > 28 files changed, 6860 insertions(+), 3 deletions(-) > > > > That's a ton of code we're adding into one of the most fragile parts of the kernel. > > > > A lot of what libqrencode does would seem to be superfluous to the requirements > > here, as we don't output kernel oopses in kanji for eg, and won't care about > > multiple versions of the qr spec. > > That's true. I didn't do that much cleanup in the library afraid of > breaking something and focused that I get this done one way or > another. Indeed, the library is userspace and is made to be versatile > rather than small. Perhaps you could add libqr to the staging tree? As long as it compiles, it can go there. Then you can focus on cleanups and bloat removal. In the process, you'll get a larger testing base because it will be in mainline. You may be interested in objdiff [1] which I'm using for merging code into the staging tree [2]. It provides an automated way to determine that code cleanups didn't change the resultant object code. You can see an example run here [3]. I would definitely like to see the QR output incorporated into a kernel.org url. That would remove the need for installing another app, and would ease bug reporting. Anyway, if you're interested, I'll be re-posting a patch for objdiff separately maybe today or this weekend. thx, Jason. [1] https://lkml.kernel.org/r/cc773270b6481ffe69516d994fbe98c13bcfdb5a.1394570067.git.jason@lakedaemon.net [2] https://lkml.kernel.org/r/cover.1394570067.git.jason@lakedaemon.net [3] https://lkml.kernel.org/r/20140312165501.GC7811@titan.lakedaemon.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/
Powered by blists - more mailing lists