Message-ID: <160b9e99-bff6-e37c-5f16-00157766535e@linux.alibaba.com>
Date:   Fri, 3 Feb 2023 17:49:33 +0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0)
 Gecko/20100101 Thunderbird/102.6.1
Subject: Re: make alloc_anon_inode more useful
Content-Language: en-US
To:     Christoph Hellwig <hch@lst.de>
Cc:     Andrew Morton <akpm@linux-foundation.org>,
        Daniel Vetter <daniel@ffwll.ch>, Nadav Amit <namit@vmware.com>,
        "VMware, Inc." <pv-drivers@vmware.com>,
        "Michael S. Tsirkin" <mst@redhat.com>,
        David Hildenbrand <david@redhat.com>,
        Minchan Kim <minchan@kernel.org>,
        Nitin Gupta <ngupta@vflare.org>,
        Jason Gunthorpe <jgg@nvidia.com>,
        Alex Williamson <alex.williamson@redhat.com>,
        linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org,
        dri-devel@lists.freedesktop.org,
        virtualization@lists.linux-foundation.org,
        linux-fsdevel@vger.kernel.org, linux-mm@kvack.org,
        Al Viro <viro@zeniv.linux.org.uk>
References: <20210309155348.974875-1-hch@lst.de>
From:   Jingbo Xu <jefflexu@linux.alibaba.com>
In-Reply-To: <20210309155348.974875-1-hch@lst.de>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi,

Sorry for digging...

This patch series seems useful for fs developers.  I'm not sure its
current status and why it doesn't get merged.


On 3/9/21 11:53 PM, Christoph Hellwig wrote:
> Hi all,
> 
> this series first renames the existing alloc_anon_inode to
> alloc_anon_inode_sb to clearly mark it as requiring a superblock.
> 
> It then adds a new alloc_anon_inode that works on the anon_inode
> file system super block, thus removing tons of boilerplate code.
> 
> The few remainig callers of alloc_anon_inode_sb all use alloc_file_pseudo
> later, but might also be ripe for some cleanup.
> 
> Diffstat:
>  arch/powerpc/platforms/pseries/cmm.c |   27 +-------------
>  drivers/dma-buf/dma-buf.c            |    2 -
>  drivers/gpu/drm/drm_drv.c            |   64 +----------------------------------
>  drivers/misc/cxl/api.c               |    2 -
>  drivers/misc/vmw_balloon.c           |   24 +------------
>  drivers/scsi/cxlflash/ocxl_hw.c      |    2 -
>  drivers/virtio/virtio_balloon.c      |   30 +---------------
>  fs/aio.c                             |    2 -
>  fs/anon_inodes.c                     |   15 +++++++-
>  fs/libfs.c                           |    2 -
>  include/linux/anon_inodes.h          |    1 
>  include/linux/fs.h                   |    2 -
>  kernel/resource.c                    |   30 ++--------------
>  mm/z3fold.c                          |   38 +-------------------
>  mm/zsmalloc.c                        |   48 +-------------------------
>  15 files changed, 39 insertions(+), 250 deletions(-)
> 

-- 
Thanks,
Jingbo