From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754121AbdLLNhC (ORCPT ); Tue, 12 Dec 2017 08:37:02 -0500 Received: from mail-wm0-f67.google.com ([74.125.82.67]:46022 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754242AbdLLMyf (ORCPT ); Tue, 12 Dec 2017 07:54:35 -0500 X-Google-Smtp-Source: ACJfBovC483uGsOPrFWOr89566P0K0dvwmLuwmWZXkeGfxkuxM1O7O297+Ux/YwQdXZmaocRnggz9LJecVfGFJd5w6w= MIME-Version: 1.0 In-Reply-To: <1513079759-14169-2-git-send-email-wei.w.wang@intel.com> References: <1513079759-14169-1-git-send-email-wei.w.wang@intel.com> <1513079759-14169-2-git-send-email-wei.w.wang@intel.com> From: Philippe Ombredanne Date: Tue, 12 Dec 2017 13:53:53 +0100 Message-ID: Subject: Re: [PATCH v19 1/7] xbitmap: Introduce xbitmap To: Wei Wang , mawilcox@microsoft.com Cc: virtio-dev@lists.oasis-open.org, LKML , qemu-devel@nongnu.org, virtualization@lists.linux-foundation.org, kvm@vger.kernel.org, linux-mm@kvack.org, mst@redhat.com, mhocko@kernel.org, akpm@linux-foundation.org, david@redhat.com, penguin-kernel@i-love.sakura.ne.jp, cornelia.huck@de.ibm.com, mgorman@techsingularity.net, aarcange@redhat.com, amit.shah@redhat.com, Paolo Bonzini , willy@infradead.org, liliang.opensource@gmail.com, yang.zhang.wz@gmail.com, quan.xu@aliyun.com, nilal@redhat.com, riel@redhat.com Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Matthew, Wei, On Tue, Dec 12, 2017 at 12:55 PM, Wei Wang wrote: > From: Matthew Wilcox > > The eXtensible Bitmap is a sparse bitmap representation which is > efficient for set bits which tend to cluster. It supports up to > 'unsigned long' worth of bits, and this commit adds the bare bones -- > xb_set_bit(), xb_clear_bit() and xb_test_bit(). > > Signed-off-by: Wei Wang > Cc: Matthew Wilcox > Cc: Andrew Morton > Cc: Michal Hocko > Cc: Michael S. Tsirkin > Cc: Tetsuo Handa [...] > --- /dev/null > +++ b/include/linux/xbitmap.h > @@ -0,0 +1,52 @@ > +/* > + * eXtensible Bitmaps > + * Copyright (c) 2017 Microsoft Corporation > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License as > + * published by the Free Software Foundation; either version 2 of the > + * License, or (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * eXtensible Bitmaps provide an unlimited-size sparse bitmap facility. > + * All bits are initially zero. > + */ Have you considered using the new SPDX ids here instead? eg. this would come out as a top line this way: > +/* SPDX-License-Identifer: GPL-2.0+ */ Overall you get less boilerplate comment and more code, so this is a win-win for everyone. This would nicely remove the legalese boilerplate with the same effect, unless you are a legalese lover of course. See Thomas doc patches for extra details.... and while you are it if you could spread the words in your team and use it for all past, present and future contributions, that would be much appreciated. And as a side benefit to me, it will help me save on paper and ink ribbons supplies for my dot matrix line printer each time I print the source code of the whole kernel. ;) Thanks for your consideration there. -- Cordially Philippe Ombredanne