From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id A4F89C4332F for ; Fri, 21 Oct 2022 13:57:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231260AbiJUN5K (ORCPT ); Fri, 21 Oct 2022 09:57:10 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41370 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231238AbiJUN5J (ORCPT ); Fri, 21 Oct 2022 09:57:09 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7CED02623CE for ; Fri, 21 Oct 2022 06:57:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1666360627; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=2q5UghdetJTCFNO+KrGeJhkMXuDAex77fkM/cP1drys=; b=aJRxBVECGvEtBa9LVYqDr372kAO+MKgOZBnTvbra52HGhVojS9i2Ey4OSQSZzbzvC/IOwY UUs7fnsSYKbKYqSTX+H5Qb//+dNrcfkbf7eAufXyOGckPL8gw8QtN1ZwESvhAoNHO0rjDI yeGlaJ2qAVJfzTDXRH3cIvmHKzYtxO4= Received: from mail-qv1-f72.google.com (mail-qv1-f72.google.com [209.85.219.72]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-595-ktxHz7GBM0u--2kjJT6vnA-1; Fri, 21 Oct 2022 09:57:06 -0400 X-MC-Unique: ktxHz7GBM0u--2kjJT6vnA-1 Received: by mail-qv1-f72.google.com with SMTP id ny15-20020a056214398f00b004b9152f4b53so2356301qvb.17 for ; Fri, 21 Oct 2022 06:57:06 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=2q5UghdetJTCFNO+KrGeJhkMXuDAex77fkM/cP1drys=; b=OW+xFYWIw/lfFLRr/gwgmsRZ5LW5sCtUosoqmYRmS7aUtuENNi4f/Cak1GbLdzivNQ DuFYJoAIN2uQBdNi5tUlUp4GBgBmqFagw4IaNO0L9tTCUYodehvIx9cxS8CF6Uju416R YQBYo99fFJ20FTtUUSJ1Um6lxAwFuaSUUK1Ee3ZTqGBkceoKbkHjFCzbU8aDaMuZg2MG grpYu1oAHlpjOeG9nboSw1k+dUe+9SRhUisMEgFnc17blAi3gip9rmLxw5nIJMsAFZme AU6GpKgVVcHQrXkZ7BaYNKN/K1Eovcx1njPsdVgpBUBr/jT5D+s8GogZxcFy9iOFamgO x1WA== X-Gm-Message-State: ACrzQf0LX/6IugBnirBn8GTvLdg2Hk75i0SkZ8lsrIivz2/EXgYueoCl xw7is4u1JjNgwib6Rc1CxlaNfB72y244NltGAl/xqDS3wt+AnwD4gmGQnXQZ0j14ok+XIR2Gqod eCs8iVlIG3F8EJNr4 X-Received: by 2002:a05:622a:4ce:b0:39c:de34:139a with SMTP id q14-20020a05622a04ce00b0039cde34139amr16940535qtx.380.1666360626004; Fri, 21 Oct 2022 06:57:06 -0700 (PDT) X-Google-Smtp-Source: AMsMyM4v459uY24feHKzdWHP9gBPSxTdJJPKDijOa3I8fTBcTkw8L/4QUAe6S/+pR5WP3uHZxL7Sag== X-Received: by 2002:a05:622a:4ce:b0:39c:de34:139a with SMTP id q14-20020a05622a04ce00b0039cde34139amr16940509qtx.380.1666360625802; Fri, 21 Oct 2022 06:57:05 -0700 (PDT) Received: from vschneid.remote.csb ([149.71.65.94]) by smtp.gmail.com with ESMTPSA id c22-20020ac85a96000000b003992448029esm8237194qtc.19.2022.10.21.06.57.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 21 Oct 2022 06:57:05 -0700 (PDT) From: Valentin Schneider To: Andy Shevchenko Cc: netdev@vger.kernel.org, linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org, Saeed Mahameed , Leon Romanovsky , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Yury Norov , Rasmus Villemoes , Ingo Molnar , Peter Zijlstra , Vincent Guittot , Dietmar Eggemann , Steven Rostedt , Mel Gorman , Greg Kroah-Hartman , Heiko Carstens , Tony Luck , Jonathan Cameron , Gal Pressman , Tariq Toukan , Jesse Brandeburg Subject: Re: [PATCH v5 2/3] sched/topology: Introduce for_each_numa_hop_mask() In-Reply-To: References: <20221021121927.2893692-1-vschneid@redhat.com> <20221021121927.2893692-3-vschneid@redhat.com> Date: Fri, 21 Oct 2022 14:57:01 +0100 Message-ID: MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On 21/10/22 16:16, Andy Shevchenko wrote: > On Fri, Oct 21, 2022 at 01:19:26PM +0100, Valentin Schneider wrote: >> The recently introduced sched_numa_hop_mask() exposes cpumasks of CPUs >> reachable within a given distance budget, wrap the logic for iterating over >> all (distance, mask) values inside an iterator macro. > > ... > >> #ifdef CONFIG_NUMA >> -extern const struct cpumask *sched_numa_hop_mask(int node, int hops); >> +extern const struct cpumask *sched_numa_hop_mask(unsigned int node, unsigned int hops); >> #else >> -static inline const struct cpumask *sched_numa_hop_mask(int node, int hops) >> +static inline const struct cpumask * >> +sched_numa_hop_mask(unsigned int node, unsigned int hops) >> { >> - if (node == NUMA_NO_NODE && !hops) >> - return cpu_online_mask; >> - >> return ERR_PTR(-EOPNOTSUPP); >> } >> #endif /* CONFIG_NUMA */ > > I didn't get how the above two changes are related to the 3rd one which > introduces a for_each type of macro. > > If you need change int --> unsigned int, perhaps it can be done in a separate > patch. > > The change inside inliner I dunno about. Not an expert. > That's a rebase fail, this should all be in the first patch, my bad.