Re: [PATCH 03/10] mm, page_alloc: Remove unnecessary taking of a seqlock when cpusets are disabled

Thu, 01 Oct 2015 00:36:15 -0700 

On 10/01/2015 12:22 AM, David Rientjes wrote:

On Mon, 21 Sep 2015, Mel Gorman wrote:

@@ -115,6 +118,9 @@ static inline unsigned int read_mems_allowed_begin(void)
   */
  static inline bool read_mems_allowed_retry(unsigned int seq)
  {
+       if (!cpusets_enabled())
+               return false;
+
        return read_seqcount_retry(&current->mems_allowed_seq, seq);
  }



I thought this was going to test nr_cpusets() <= 1?
That was another patch in prior iteration of the series, but turns out it was unnecessary, because cpusets_enabled() is already only true when nr_cpusets() > 1 - see https://lkml.org/lkml/2015/8/25/300 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to