On Wed, Mar 16, 2022 at 2:44 PM Mahmoud Abumandour <ma.mando...@gmail.com>
wrote:

>
>
> On Wed, Mar 16, 2022 at 2:40 PM Christoph Muellner <cmuell...@linux.com>
> wrote:
>
>> QEMU plugins can be loaded via command line arguments or via
>> the QEMU_PLUGIN environment variable. Currently, only the first method
>> is documented. Let's document QEMU_PLUGIN.
>>
>> Signed-off-by: Christoph Muellner <cmuell...@linux.com>
>> ---
>>  docs/devel/tcg-plugins.rst | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/docs/devel/tcg-plugins.rst b/docs/devel/tcg-plugins.rst
>> index f93ef4fe52..ba48be18d0 100644
>> --- a/docs/devel/tcg-plugins.rst
>> +++ b/docs/devel/tcg-plugins.rst
>> @@ -34,6 +34,10 @@ Arguments are plugin specific and can be used to
>> modify their
>>  behaviour. In this case the howvec plugin is being asked to use inline
>>  ops to count and break down the hint instructions by type.
>>
>> +QEMU also evaluates the environment variable ``QEMU_PLUGIN``::
>> +
>> +  QEMU_PLUGIN="file=tests/plugin/libhowec.so,inline=on,count=hint" $QEMU
>>
>
> The plugin howvec is in contrib/plugins, so I think giving the correct
> path would be
> better. Note also that there's a typo in "libhowec.so".
>

Oh yes, I was just copying the contents of the example above but did not
notice
that this is also wrong.

I'll respin a v2 which fixes the paths.

Thanks!


>
> If you want, you could also fix the next example that has the same path or
> leave it
> for another patch.
>
> +
>>  Writing plugins
>>  ---------------
>>
>> --
>> 2.35.1
>>
>>
> Other than that,
> Reviewed-By: Mahmoud Mandour <ma.mando...@gmail.com>
>
> Thanks,
> Mahmoud
>

Reply via email to