From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Zijlstra Subject: Re: [PATCH v4 01/11] cpumask: factor out show_cpumap into separate helper function Date: Thu, 4 Sep 2014 13:25:51 +0200 Message-ID: <20140904112551.GC346@worktop.programming.kicks-ass.net> References: <1409763617-17074-1-git-send-email-sudeep.holla@arm.com> <1409763617-17074-2-git-send-email-sudeep.holla@arm.com> <20140904062006.GC3190@worktop.ger.corp.intel.com> <54082ADD.6040004@arm.com> <20140904092135.GA346@worktop.programming.kicks-ass.net> <54084246.7010705@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from bombadil.infradead.org ([198.137.202.9]:36887 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753276AbaIDL0F (ORCPT ); Thu, 4 Sep 2014 07:26:05 -0400 Content-Disposition: inline In-Reply-To: <54084246.7010705@arm.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Sudeep Holla Cc: LKML , Heiko Carstens , Lorenzo Pieralisi , Greg Kroah-Hartman , "Rafael J. Wysocki" , Bjorn Helgaas , "x86@kernel.org" , "linux-acpi@vger.kernel.org" , "linux-pci@vger.kernel.org" On Thu, Sep 04, 2014 at 11:43:18AM +0100, Sudeep Holla wrote: > >You could make it bitmap_copy_to_buf() I suppose, and then have a > >cpumask inline wrapper, to be fully consistent with the rest of the > >implementations. > > > > I liked the second approach, how about patch below ? > If will munge it into original patch. Looks good, one more nit if you will permit :-) > +int bitmap_copy_to_pagebuf(bool list, const unsigned long *maskp, char *buf) Maybe s/copy/print/ or something? Its not really a 'copy' as such since it very much transforms the bitmap.