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
>
>
>