Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753157AbcLHLL0 (ORCPT ); Thu, 8 Dec 2016 06:11:26 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:55176 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751396AbcLHLLZ (ORCPT ); Thu, 8 Dec 2016 06:11:25 -0500 Date: Thu, 8 Dec 2016 12:02:29 +0100 From: Greg Kroah-Hartman To: Rasmus Villemoes Cc: Jens Axboe , Tejun Heo , Andrew Morton , linux-kernel@vger.kernel.org, Lai Jiangshan , linux-block@vger.kernel.org Subject: Re: [RFC 06/10] block: use tida as small id allocator Message-ID: <20161208110229.GA28572@kroah.com> References: <1481160187-9652-1-git-send-email-linux@rasmusvillemoes.dk> <1481160187-9652-7-git-send-email-linux@rasmusvillemoes.dk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.7.2 (2016-11-26) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 961 Lines: 22 On Wed, Dec 07, 2016 at 08:56:27PM -0700, Jens Axboe wrote: > On 12/07/2016 06:23 PM, Rasmus Villemoes wrote: > > A struct ida ends up costing > 16 KB of runtime memory, which is quite > > a lot for something which on my laptop as of this writing has handed > > out 27 ids in its lifetime. So use the simpler and lighter-weight > > struct tida. > > I'm worried that your example of your laptop isn't an all encompassing > test case. How well does the simplified ida allocator work for tens of > thousands of disks, at scan time? SCSI is notorious for setting up and > tearing down a ton of queues at probe time. > > Unless we have more testing than 'it works on my laptop and saves 16k', > I'm not super intereted in the patch. Rasmus, you can create 30k virtual scsi devices on your laptop to test if this really does work or saves any real time or memory. I'd recommend doing that if you want to get patches like this ever accepted. good luck! greg k-h