On Aug 11 2024, at 3:51 pm, Michael Tokarev <m...@tls.msk.ru> wrote:
> 12.06.2024 15:43, Amjad Alsharafi wrote:
>> These patches fix some bugs found when modifying files in vvfat.
>> First, there was a bug when writing to the cluster 2 or above of a
>> file, it
>> will copy the cluster before it instead, so, when writing to
>> cluster=2, the
>> content of cluster=1 will be copied into disk instead in its place.
>>
>> Another issue was modifying the clusters of a file and adding new
>> clusters, this showed 2 issues:
>> - If the new cluster is not immediately after the last cluster, it will
>> cause issues when reading from this file in the future.
>> - Generally, the usage of info.file.offset was incorrect, and the
>> system would crash on abort() when the file is modified and a new
>> cluster was added.
>>
>> Also, added some iotests for vvfat, covering the this fix and also
>> general behavior such as reading, writing, and creating files on the
>> filesystem.
>> Including tests for reading/writing the first cluster which
>> would pass even before this patch.
> ...
>> Amjad Alsharafi (5):
>> vvfat: Fix bug in writing to middle of file
>> vvfat: Fix usage of `info.file.offset`
>> vvfat: Fix wrong checks for cluster mappings invariant
>> vvfat: Fix reading files with non-continuous clusters
>> iotests: Add `vvfat` tests
>
> Actually, maybe the whole series is a good candidate for -stable, not
> just the
> first fix. What do you think?
>
> Thanks,
>
> /mjt
Hello Michael,
This actually has been reviewed and approved (last version was v6 here:
https://patchew.org/QEMU/cover.1721470238.git.amjadsharaf...@gmail.com/)
It has been merged into upstream here:
https://gitlab.com/qemu-project/qemu/-/commit/6d00c6f982562222adbd0613966285792125abe5
Or do you perhaps mean something else by `-stable`?
Thanks,
Amjad
>
> --
> GPG Key transition (from rsa2048 to rsa4096) since 2024-04-24.
> New key: rsa4096/61AD3D98ECDF2C8E 9D8B E14E 3F2A 9DD7 9199 28F1 61AD
> 3D98 ECDF 2C8E
> Old key: rsa2048/457CE0A0804465C5 6EE1 95D1 886E 8FFB 810D 4324 457C
> E0A0 8044 65C5
> Transition statement: http://www.corpit.ru/mjt/gpg-transition-2024.txt
>
>
