Date:   Thu, 16 Mar 2023 17:03:52 +0500
From:   Khadija Kamran <kamrankhadijadj@gmail.com>
To:     "Fabio M. De Francesco" <fmdefrancesco@gmail.com>
Cc:     Alison Schofield <alison.schofield@intel.com>,
        outreachy@lists.linux.dev,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org,
        Nathan Chancellor <nathan@kernel.org>
Subject: Re: [PATCH] staging: axis-fifo: initialize timeouts in probe only
Message-ID: <ZBMFqDWuZQdg3YP3@khadija-virtual-machine>
References: <ZA9mThZ7NyRrQAMX@khadija-virtual-machine>
 <3096768.uvML7RA1vL@suse>
 <ZBLmev7BaIhYQXIz@khadija-virtual-machine>
 <1797409.3VsfAaAtOV@suse>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1797409.3VsfAaAtOV@suse>
Precedence: bulk

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