Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 4 Jan 2023 11:50:29 -0800
From:   Luis Chamberlain <mcgrof@kernel.org>
To:     Dan Williams <dan.j.williams@intel.com>
Cc:     alison.schofield@intel.com, vishal.l.verma@intel.com,
        ira.weiny@intel.com, bwidawsk@kernel.org, dave@stgolabs.net,
        a.manzanares@samsung.com, linux-cxl@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] cxl_test: remove nvdimm requirement
Message-ID: <Y7XYhYdf0NEEdZAZ@bombadil.infradead.org>
References: <20221219195620.351544-1-mcgrof@kernel.org>
 <63a21774a3e03_1bf2294bc@dwillia2-xfh.jf.intel.com.notmuch>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <63a21774a3e03_1bf2294bc@dwillia2-xfh.jf.intel.com.notmuch>
Sender: Luis Chamberlain <mcgrof@infradead.org>
Precedence: bulk

On Tue, Dec 20, 2022 at 12:13:40PM -0800, Dan Williams wrote:
> Luis Chamberlain wrote:
> > There is no need for NVDIMM to test CXL, so remove this
> > requirement.
> 
> Are you perhaps running without the new nvdimm pmem security test?
> 
> http://lore.kernel.org/r/167105505204.3034751.8113387624258581781.stgit@djiang5-desk3.ch.intel.com/
> 
> ...again these types of mismatches are expected in and around the merge
> window when linux.git and ndctl.git get out of sync. I would skip
> running cxl_test for regression purposes until around -rc2 timeframe
> when all of the new code has settled in both repositories.

No, since one can build CXL without NVDIMM it seems this is a fair
change as well.

  Luis