Re: [PATCH] staging: axis-fifo: initialize timeouts in probe only

From: Khadija Kamran
Date: Thu Mar 16 2023 - 08:04:01 EST


On Thu, Mar 16, 2023 at 12:13:23PM +0100, Fabio M. De Francesco wrote:
> Khadija,
>
> I'm not yet sure whether or not you are doing well with reserving 13GB to a
> VM. First of all you should better use multiples of 2GB. Furthermore it looks
> too much memory unless you have a total "real" RAM in your host exceeding
> 18GB. You must be sure you are leaving enough room for the Windows host to run
> smoothly and avoid too much swapping to/from disk.
>

Hey Fabio,

I have 16GB RAM. I just chose the highest option that VMware
recommended. My Windows is running smoothly.

> At the same time I doubt that you can reserve 8 logical cores (2x4) for
> running the VM on VMWare.
>
> However, we have time to investigate and adjust your configuration (if it
> really needs to be fine tuned). I'll try to reach you on IRC, open a private
> window, and assist with this task ASAP.
>
> At the moment I'd prefer to see you focusing on getting your first patch
> accepted by Greg.
>

I am working on the new [PATCH v5] as I was able to remove the warnings.
However, If you still think that we should work on the configurations
then we can start working on it. :)

Thank you for all the help.

Regards,
Khadija



> Thanks,
>
> Fabio
>
>
>