Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <1c45a3aa-275a-2815-79bf-ef72df30db97@paragon-software.com>
Date:   Fri, 30 Dec 2022 16:07:12 +0400
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.0
Subject: Re: [PATCH 0/3] fs/ntfs3: Fixes for big endian systems
Content-Language: en-US
To:     =?UTF-8?Q?Thomas_K=c3=bchnel?= <thomas.kuehnel@avm.de>
CC:     <ntfs3@lists.linux.dev>, <linux-kernel@vger.kernel.org>
References: <20211207102454.576906-1-thomas.kuehnel@avm.de>
From:   Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
In-Reply-To: <20211207102454.576906-1-thomas.kuehnel@avm.de>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk

On 07.12.2021 14:24, Thomas Kühnel wrote:
> I tried running the NTFS3 driver on a MIPS big endian system and
> noticed various errors when it tried to access bitmaps structures
> that are stored as little endian on disk.
>
> These patches were mainly tested in a qemu mips environment.
>
> I'm open for suggestions how to better implement the second patch
> "add functions to modify LE bitmaps". It adds copies of two functions
> from lib/bitmap.c but modified to work with little endian bitmaps.
> Other filesystems seem to have similar functions defined locally like
> le_bitmap_set in btrfs or ext4_set_bits in ext4 but no global
> implementation exists.
>
> Thomas Kühnel (3):
>    fs/ntfs3: fix endian conversion in ni_fname_name
>    fs/ntfs3: add functions to modify LE bitmaps
>    fs/ntfs3: use _le variants of bitops functions
>
>   fs/ntfs3/bitmap.c  | 56 +++++++++++++++++++++++++++++++++++++++-------
>   fs/ntfs3/frecord.c |  4 +++-
>   fs/ntfs3/fslog.c   |  4 ++--
>   fs/ntfs3/fsntfs.c  |  8 +++----
>   fs/ntfs3/index.c   | 14 ++++++------
>   fs/ntfs3/ntfs_fs.h |  3 +++
>   6 files changed, 67 insertions(+), 22 deletions(-)
>
>
> base-commit: 0fcfb00b28c0b7884635dacf38e46d60bf3d4eb1

Hello Thomas,

Thanks for patches, they have been applied long ago.

Sorry for not responding immediately.