Ping^3 for review (or I'm going to just apply these anyway).

Paolo, I know we discussed this on IRC, but can I get an Acked-by
tag from you for the license clarification in patch 6, please?

thanks
-- PMM

On Thu, 19 Sept 2024 at 13:12, Peter Maydell <peter.mayd...@linaro.org> wrote:
>
> Ping^2.
>
> In particular: Paolo, it would be helpful if you could
> confirm what license you intended for the lockcnt.c code.
>
> thanks
> -- PMM
>
> On Fri, 6 Sept 2024 at 16:12, Peter Maydell <peter.mayd...@linaro.org> wrote:
> >
> > Ping for review on these docs conversions, please?
> >
> > thanks
> > -- PMM
> >
> > On Fri, 16 Aug 2024 at 14:22, Peter Maydell <peter.mayd...@linaro.org> 
> > wrote:
> > >
> > > This patchset converts a collection of the remaining txt files in
> > > docs/devel to rST format. At this point in the release cycle
> > > I'm targeting 9.2 for these.
> > >
> > > Patches 1-5 are simple conversions.  In patch 6 I move the
> > > QemuLockCnt APIs to their own header file; this is so that we can
> > > include the kernel-doc generated APIs in lockcnt.rst (since
> > > kernel-doc works on a per-file basis).
> > >
> > > Paolo: can you confirm that you mean GPLv2-or-later for the lockcnt
> > > code?  There's no explicit license comment in either util/lockcnt.c
> > > or include/qemu/thread.h.
> > >
> > > (There's one .txt file left in docs/devel after this, but I don't
> > > think it's very useful so I propose to delete it and will send a
> > > separate patch for that.)
> > >
> > > thanks
> > > -- PMM
> > >
> > > Peter Maydell (7):
> > >   docs/devel/blkdebug: Convert to rST format
> > >   docs/devel/blkverify: Convert to rST format
> > >   docs/devel/lockcnt: Convert to rST format
> > >   docs/devel/multiple-iothreads: Convert to rST format
> > >   docs/devel/rcu: Convert to rST format
> > >   include: Move QemuLockCnt APIs to their own header
> > >   docs/devel/lockcnt: Include kernel-doc API documentation
> > >
> > >  MAINTAINERS                                 |   8 +-
> > >  docs/devel/blkdebug.rst                     | 177 ++++++++++++++++++++
> > >  docs/devel/blkdebug.txt                     | 162 ------------------
> > >  docs/devel/{blkverify.txt => blkverify.rst} |  30 ++--
> > >  docs/devel/index-api.rst                    |   1 +
> > >  docs/devel/index-build.rst                  |   2 +
> > >  docs/devel/index-internals.rst              |   2 +
> > >  docs/devel/{lockcnt.txt => lockcnt.rst}     |  89 +++++-----
> > >  docs/devel/multiple-iothreads.rst           | 139 +++++++++++++++
> > >  docs/devel/multiple-iothreads.txt           | 130 --------------
> > >  docs/devel/{rcu.txt => rcu.rst}             | 172 +++++++++----------
> > >  include/block/aio.h                         |   1 +
> > >  include/hw/core/cpu.h                       |   1 +
> > >  include/qemu/lockcnt.h                      | 130 ++++++++++++++
> > >  include/qemu/thread.h                       | 111 ------------
> > >  accel/accel-blocker.c                       |   1 +
> > >  hw/core/cpu-common.c                        |   1 +
> > >  util/aio-posix.c                            |   1 +
> > >  util/aio-win32.c                            |   1 +
> > >  util/async.c                                |   1 +
> > >  util/fdmon-epoll.c                          |   1 +
> > >  util/lockcnt.c                              |   1 +
> > >  22 files changed, 608 insertions(+), 554 deletions(-)
> > >  create mode 100644 docs/devel/blkdebug.rst
> > >  delete mode 100644 docs/devel/blkdebug.txt
> > >  rename docs/devel/{blkverify.txt => blkverify.rst} (77%)
> > >  rename docs/devel/{lockcnt.txt => lockcnt.rst} (75%)
> > >  create mode 100644 docs/devel/multiple-iothreads.rst
> > >  delete mode 100644 docs/devel/multiple-iothreads.txt
> > >  rename docs/devel/{rcu.txt => rcu.rst} (73%)
> > >  create mode 100644 include/qemu/lockcnt.h

Reply via email to