linux-kernel - Re: scsi: Implement per-cpu logging buffer

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: <54DCAABF.20606@suse.de>
Date:	Thu, 12 Feb 2015 14:29:35 +0100
From:	Hannes Reinecke <hare@...e.de>
To:	Geert Uytterhoeven <geert@...ux-m68k.org>,
	Christoph Hellwig <hch@....de>
CC:	Josh Triplett <josh@...htriplett.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	scsi <linux-scsi@...r.kernel.org>
Subject: Re: scsi: Implement per-cpu logging buffer

On 02/12/2015 01:36 PM, Geert Uytterhoeven wrote:
> On Wed, Feb 11, 2015 at 8:16 PM, Linux Kernel Mailing List
> <linux-kernel@...r.kernel.org> wrote:
>> Gitweb:     http://git.kernel.org/linus/;a=commit;h=ded85c193a391a84076d5c6a7a5668fe164a490e
>> Commit:     ded85c193a391a84076d5c6a7a5668fe164a490e
>> Parent:     b0a93d96b2814c725161f91a4e35d0c29ec0f95b
>> Refname:    refs/heads/master
>> Author:     Hannes Reinecke <hare@...e.de>
>> AuthorDate: Thu Jan 8 07:43:42 2015 +0100
>> Committer:  Christoph Hellwig <hch@....de>
>> CommitDate: Fri Jan 9 15:44:28 2015 +0100
>>
>>     scsi: Implement per-cpu logging buffer
>>
>>     Implement a per-cpu buffer for formatting messages to avoid line breaks
>>     up under high load.  This patch implements scmd_printk() and
>>     sdev_prefix_printk() using the per-cpu buffer and makes sdev_printk() a
>>     wrapper for sdev_prefix_printk().
>>
>>     Tested-by: Robert Elliott <elliott@...com>
>>     Reviewed-by: Robert Elliott <elliott@...com>
>>     Signed-off-by: Hannes Reinecke <hare@...e.de>
>>     Signed-off-by: Christoph Hellwig <hch@....de>
> 
>> --- /dev/null
>> +++ b/drivers/scsi/scsi_logging.c
> 
>> +#define SCSI_LOG_SPOOLSIZE 4096
>> +#define SCSI_LOG_BUFSIZE 128
>> +
>> +#if (SCSI_LOG_SPOOLSIZE / SCSI_LOG_BUFSIZE) > BITS_PER_LONG
>> +#warning SCSI logging bitmask too large
>> +#endif
>> +
>> +struct scsi_log_buf {
>> +       char buffer[SCSI_LOG_SPOOLSIZE];
>> +       unsigned long map;
>> +};
>> +
>> +static DEFINE_PER_CPU(struct scsi_log_buf, scsi_format_log);
> 
> Do we really need a static 4 KiB per-CPU buffer?
> 
> bloat-o-meter:
> 
> add/remove: 183/94 grow/shrink: 314/211 up/down: 33467/-21291 (12176)
> function                                     old     new   delta
> scsi_format_log                                -    4100   +4100
> handle_mm_fault                             1794    2750    +956
> scsi_log_print_sense_hdr                       -     774    +774
> proc_keys_show                                 -     770    +770
> 
Define 'need'.
We don't absolutely 'need' it. (Configure it out and it's gone).

But when we want to avoid several logging messages coming in from
various CPUs overwriting each other and _not_ introduce additional
latency by locking a single buffer, then yes.

We can possibly reduce it to, say, 1KiB or even lower by imposing
stricter rules on the logging functions.
But I don't see a way around the per-CPU buffer.

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		               zSeries & Storage
hare@...e.de			               +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)
--
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/