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;
Message-ID: <53c19bba-4513-0778-5e9a-2cffddad4465@intel.com>
Date:   Thu, 3 Aug 2023 18:42:12 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.13.0
Subject: Re: [PATCH net-next 5/9] page_pool: don't use driver-set flags field
 directly
Content-Language: en-US
To:     Jakub Kicinski <kuba@kernel.org>
CC:     Yunsheng Lin <linyunsheng@huawei.com>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Paolo Abeni <pabeni@redhat.com>,
        Maciej Fijalkowski <maciej.fijalkowski@intel.com>,
        Larysa Zaremba <larysa.zaremba@intel.com>,
        Alexander Duyck <alexanderduyck@fb.com>,
        Jesper Dangaard Brouer <hawk@kernel.org>,
        "Ilias Apalodimas" <ilias.apalodimas@linaro.org>,
        Simon Horman <simon.horman@corigine.com>,
        <netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>
References: <20230727144336.1646454-1-aleksander.lobakin@intel.com>
 <20230727144336.1646454-6-aleksander.lobakin@intel.com>
 <a0be882e-558a-9b1d-7514-0aad0080e08c@huawei.com>
 <6f8147ec-b8ad-3905-5279-16817ed6f5ae@intel.com>
 <a7782cf1-e04a-e274-6a87-4952008bcc0c@huawei.com>
 <0fe906a2-5ba1-f24a-efd8-7804ef0683b6@intel.com>
 <20230802142920.4a777079@kernel.org>
 <7b77dd3a-fd03-884a-8b8a-f76ab6de5691@intel.com>
 <20230803090029.16a6798d@kernel.org>
 <7c1c6ec2-90cb-84e0-1bc3-cf758f15e384@intel.com>
 <20230803094043.31d27cb5@kernel.org>
From:   Alexander Lobakin <aleksander.lobakin@intel.com>
In-Reply-To: <20230803094043.31d27cb5@kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?eDFGZEhaWXVSanRoQnlGTVNiWVo2TGw0ME4xdkJZMjJxdzlpY0ZrUVBpK3RP?=
 =?utf-8?B?dG84UHRoN210MlBwSnJtU1Qwc1IyOGoreXhsckhtVDJVbnhxNjJxemlwWXhC?=
 =?utf-8?B?OG1XeUlsUFF0c25OZzRFT2lYNndvUHpieWNpcit4ZXhXUFlSTmdyNTcyS3lR?=
 =?utf-8?B?bHMzNG1OVlBEMlNTSTB5Z0hFSW5uc1Jjek45Rk52bHdGbFY5dERzdzA4N3BO?=
 =?utf-8?B?eEZZb1dnclJpMDJjM1FObGlweHh6QjFQUWdSZmdsb2NEQzBSaEJvdS9mSzJG?=
 =?utf-8?B?R1Byb0tncWZvQUpEczhsTVhPTlMvcUVVUGo1RkNEYWZZMHpkZ1Zac2YrRFl5?=
 =?utf-8?B?elpZMnRDRjFYMUwzQmRoNmc2MDhCY1Y4KzRYMnVSTXZTSGs5S2VzRU9vTC9B?=
 =?utf-8?B?aW90ekJDd0JDdXVMSUFZS3psZHJvd2syNnhISGNJMWVmenl3UXdUSmxBN1Jx?=
 =?utf-8?B?d1VTclRqTXRDemRqcnZBUDhoODVTNXlCemRWM0tsN2JVU0RLU0RGbFJVeC9m?=
 =?utf-8?B?NFcxc1A0VTBENmRQQTN2ZHNGV1hNQkp0b0YzMm9XQ2xRMko2RXBkVTRvQ1Jm?=
 =?utf-8?B?cEhVN042cU1LcTlHaEp4MWorY1NSVFdWUXNLSmhxK1BFL0NBWW9BMXNTTkIw?=
 =?utf-8?B?REJjeVZzYmFDbzc4SE1VU1Z1am1GYnZ3NEk4OGp0ZlNJaG44WkFYbklBNGlG?=
 =?utf-8?B?eFhBYVVoSTZlNzdhaXpGbXVyUUxEVDlMNE1WMGl4WDJFL05qTFhxMWF4Q0Rt?=
 =?utf-8?B?QlFLclpFbUlwZC8vL3ZadjRzUG5nS1ZpM1R0UVpuVzAyR1RUeFhTOUpUc2Yw?=
 =?utf-8?B?OXM1bWtVSkplTU5MYTF0dktzUWVyTGRxUUhsc0xBcS9IZTNPcTdVUGNYRUdD?=
 =?utf-8?B?RlVmdVQyWDAvclQ1cUE5b1BreTc1K3JFOWxGbGF1cUs2bzJkdmpiVGhUbmRG?=
 =?utf-8?B?ZFVid2ZDdmswNDRpdDV5cDhmVElxazBuZUtKbzJoeHg4ejhBQXFaYjUrNlhZ?=
 =?utf-8?B?RU9LdVErellFM0tsRDFGNHdrSmt4ZkVsYnlKQWhtVFNQdDdsTXoxcGRUREJ0?=
 =?utf-8?B?eHhKcUpIMTZhSEdudkg1VlZHT1Z4dXpRTE9uMVdsSFlGVXRPb2xvb2RrRENW?=
 =?utf-8?B?d3M2eW45NU52cCtFait1M1lBcisydURYb1d2M2NUdXpaMnAxYmhmR2xRcXBo?=
 =?utf-8?B?MStJNGpzVEduaU5nUmFBdzV4NVYrcUlvYlk0Nms4N0ZyTUVRNThmNHRQV1cx?=
 =?utf-8?B?d0hSSkkrTGxCUFBOVjVQYlhkTEJFNDdCMlJRWGlrZUpJM1I3UXBLdjZReDlo?=
 =?utf-8?B?ZWh6VlZMalJBWC9OMVBsSThrNDJIcVoyTDNsNUhBdTI5VkEvakl1SzBUd3Bq?=
 =?utf-8?B?dDZFbG5haDJmR2tPWGdYZWVCY1ZOOFBOMXpqT1kybkNGK0RVeHR0UU9EZUY2?=
 =?utf-8?B?eCtvTDYra1lGd1pBUUc4YmkxdXNmdG5kd25adklLL3NGeWJJY2VFL0tBZnVO?=
 =?utf-8?B?N3RsbHVSdkZ3RjNRQ0ExYkNNYVJiWVFKckQwTWVHUW02YzZvOWZhY0d3c0ww?=
 =?utf-8?B?NTZUTmpGVG9IYXNaaFNxVkU4NkJmZjZVZlpvU3I0KzlvRHlUSE9CYVhId0h2?=
 =?utf-8?B?ZGNQSzZ3dGVLc3EyRXdTbVpKd1UzZy9LQld3ckdxQU9meGZQZ3ZZMkl4cVJP?=
 =?utf-8?B?Tno5R1FCSndxNENLLzVTVUR4dFpLdkRBamphQXI3cVVYeHI4blZZWldicXRn?=
 =?utf-8?B?blVkT0JlK2hUaWZyY0tFOEpsTllvZzZHRmM4d1hIS2tqSzNsOE0zakdrcjVN?=
 =?utf-8?B?TWJpUjFCNGdZQ0NvSWIvWm5DL0hMSGRrS3lzYW1wb1pyYWNqMUFwWks1cllE?=
 =?utf-8?B?TVRwU3laMDRPcTgwTU5xaS9pS1BoNGdTcHp0QWV2QzhEbFlYbE0rdzJjcDZQ?=
 =?utf-8?B?TzBjd25GV0dPSVNWUmxsd2xYcTg2a3htOGptS0tLVktONjVFRS9FYmgvSnJX?=
 =?utf-8?B?ZjAwLzJ0MHdYc1hGSHFzVmRyZ2tOWnNRbU5pYlJsZ3FiRUxMcWkzcUNJRmpD?=
 =?utf-8?B?MlMzOG1YUElHNzVnc01KckcrUzBHNytuUVJJU3J2T0R5OGQyNG0rK3JNbU52?=
 =?utf-8?B?Uk93NzZYV0NBTUF1V1Z2MTh5a0pPbS9wMmNuKzVvRVR0UEE1WEZad0kza3F3?=
 =?utf-8?B?dHc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: b0048768-8993-494e-252d-08db94409a6c
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB3625.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Aug 2023 16:42:19.4882
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: j2Qc9dNGnp91j7g/TncZXmEc3dPGL5GaNvpU2VUKR0f6FUTIPEYoZtL4708/jbawXk/qd/4PwbAmpiNvRwGZczeNgAR46sYByUZIamzQrR0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR11MB6865
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-2.2 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

From: Jakub Kicinski <kuba@kernel.org>
Date: Thu, 3 Aug 2023 09:40:43 -0700

> On Thu, 3 Aug 2023 18:07:23 +0200 Alexander Lobakin wrote:
>>>> I would propose to include it in the series, but it has grown a bunch
>>>> already and it's better to do that later separately :s  
>>>
>>> Yeah.. I'd be trying to split your series up a little to make progress
>>> rather than add more things :( I was going to suggest that you post
>>> just the first 3 patches for instance. Should be an easy merge.  
>>
>> One minute before I was going to post v2 :>
>> Sounds good. AFACS only #4-6 are still under question (not for me tho
>> :D), let me move that piece out.
> 
> FWIW I'll apply my doc changes in the next 5min if nobody objects.
> Can you rebase on top of that?

Breh, sent already >_<

What do I do then? Rebase and send v3?

Thanks,
Olek