[Qemu-devel] [PATCH v2 0/4] 9p: Fix file ID collisions

[Qemu-devel] [PATCH v2 0/4] 9p: Fix file ID collisions

[Christian Schoenebeck]

Hi!

This is v2 of a proposed patch set for fixing file ID collisions with 9pfs.

Patch 1 to 3 are identical to the previous version. New in this v2 is patch 4
which introduces variable length suffixes for inode mapping instead of fixed
length prefixes.

Also: patch 4 disables file ID persistency at compile time by default for now,
since I am yet unresolved about details of that persistency.

Christian Schoenebeck (4):
  9p: mitigates most QID path collisions
  9P: trivial cleanup of QID path collision mitigation
  9p: persistency of QID path beyond reboots / suspensions
  9p: use variable length suffixes for inode mapping

 fsdev/9p-marshal.h   |    6 +-
 hw/9pfs/9p.c         | 1145 ++++++++++++++++++++++++++++++++++++++++++++++++--
 hw/9pfs/9p.h         |  167 ++++++++
 hw/9pfs/trace-events |   14 +-
 4 files changed, 1296 insertions(+), 36 deletions(-)

-- 
2.11.0

[Qemu-devel] [PATCH v2 0/4] 9p: Fix file ID collisions

[Christian Schoenebeck via Qemu-devel]

Hi!

This is v2 of a proposed patch set for fixing file ID collisions with 9pfs.

Patch 1 to 3 are identical to the previous version. New in this v2 is patch 4
which introduces variable length suffixes for inode mapping instead of fixed
length prefixes.

Also: patch 4 disables file ID persistency at compile time by default for now,
since I am yet unresolved about details of that persistency.

Christian Schoenebeck (4):
  9p: mitigates most QID path collisions
  9P: trivial cleanup of QID path collision mitigation
  9p: persistency of QID path beyond reboots / suspensions
  9p: use variable length suffixes for inode mapping

 fsdev/9p-marshal.h   |    6 +-
 hw/9pfs/9p.c         | 1145 ++++++++++++++++++++++++++++++++++++++++++++++++--
 hw/9pfs/9p.h         |  167 ++++++++
 hw/9pfs/trace-events |   14 +-
 4 files changed, 1296 insertions(+), 36 deletions(-)

-- 
2.11.0

Re: [Qemu-devel] [PATCH v2 0/4] 9p: Fix file ID collisions

[Greg Kurz]

On Fri, 03 May 2019 18:20:39 +0200
Christian Schoenebeck <qemu_oss@crudebyte.com> wrote:

> Hi!
> 
> This is v2 of a proposed patch set for fixing file ID collisions with 9pfs.
> 
> Patch 1 to 3 are identical to the previous version. New in this v2 is patch 4
> which introduces variable length suffixes for inode mapping instead of fixed
> length prefixes.
> 
> Also: patch 4 disables file ID persistency at compile time by default for now,
> since I am yet unresolved about details of that persistency.
> 
> Christian Schoenebeck (4):
>   9p: mitigates most QID path collisions
>   9P: trivial cleanup of QID path collision mitigation
>   9p: persistency of QID path beyond reboots / suspensions
>   9p: use variable length suffixes for inode mapping
> 
>  fsdev/9p-marshal.h   |    6 +-
>  hw/9pfs/9p.c         | 1145 ++++++++++++++++++++++++++++++++++++++++++++++++--
>  hw/9pfs/9p.h         |  167 ++++++++
>  hw/9pfs/trace-events |   14 +-
>  4 files changed, 1296 insertions(+), 36 deletions(-)
> 

Hmm... this series comes unthreaded again because the cover has:

Message-ID: <1926077.q6xKvCkoZ9@silver>

while the patches have:

In-Reply-To: <cover.1556896526.git.qemu_oss@crudebyte.com>

Maybe consider using git send-email or stgit mail to have proper
threading for free.

Cheers,

--
Greg