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:   Tue, 6 Sep 2022 18:08:04 +0200
From:   =?utf-8?Q?Micha=C5=82?= Winiarski <michal.winiarski@intel.com>
To:     Matthew Wilcox <willy@infradead.org>
CC:     <dri-devel@lists.freedesktop.org>, <linux-kernel@vger.kernel.org>,
        "David Airlie" <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>,
        "Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>,
        Maxime Ripard <mripard@kernel.org>,
        Thomas Zimmermann <tzimmermann@suse.de>,
        Simon Ser <contact@emersion.fr>
Subject: Re: [PATCH v3 0/4] drm: Use full allocated minor range for DRM
Message-ID: <20220906160804.wvuioiya33df7jnj@nostramo>
References: <20220906140117.409852-1-michal.winiarski@intel.com>
 <YxdXZbFBHIevYwRz@casper.infradead.org>
Content-Type: text/plain; charset="utf-8"
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <YxdXZbFBHIevYwRz@casper.infradead.org>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?dnl3Z1EveWdsZ2ZHU040SW12K2dJRk1nOXRockk4cUcvMStpcnNqczZ3N0U2?=
 =?utf-8?B?U3FoZ2I2KzNaWnRaSFNUY29GWXJpQml4OTBMTTVaTURVeWNtb0puL1Z0Q1Qr?=
 =?utf-8?B?cW5jTDJaZ25Ua0dncGVIeCsxRzAxMWx2ODBWUzFka0VxWFN0bWhneTlaSmZP?=
 =?utf-8?B?dWRkcnRlb3pkQXRzT29uZTdMUzY0YWRCdEpFVlhjV1N1dW0xV0ZVSTc1YTVx?=
 =?utf-8?B?b1hIcjdMQUVrdDlWY0QveVRWUVpwSlc4RGZZbGZhVzlLa2UwZlFmYUNKdS94?=
 =?utf-8?B?WUZwMVZMU1pLdW1nZFVTMVR6Y1U3L2VLWnFBOXVYYWdSMFZlZWlNZ2JsekRa?=
 =?utf-8?B?Z094WjAwekxUc0haTE15TDUzcEJXZUJkMmlSbXpOMTEybXdnUDkxTHByOHl0?=
 =?utf-8?B?ckN2STB5VktBS1lkbGg5Ly9rSWc1eEtHVi9pd0ViVFRJZ3RFMWhtaEtwT1NK?=
 =?utf-8?B?RTU4M21hQUwrN3c3R1B4RXZMT1QyZi9Oc2hjMXlYdjhjU3lkbjd5QjRBVWJM?=
 =?utf-8?B?SDNMbUxuYlpXUlhSR1ZuY2NZb0x1R2dwaCszU0gvRitrNVNFWjlmUTE3cmYr?=
 =?utf-8?B?ZkZrOTA1TDNOa2ZxQkJFcmNlK3VnM0lYRlNUTzlxWHdmazNHd1lYdEVwU0Er?=
 =?utf-8?B?WDYwWVphTzh6NTJnQ2xVdldmNjNsdXJQUnJZT05pUllLN3Nzb0Q4Wjk1QUpa?=
 =?utf-8?B?UkZWMHZUa05qWTNzTGpqQnp0b05PaGF0VmJlV2FURjk5MnhQL3VQd1puZlNs?=
 =?utf-8?B?SVpNSXRTSTB1RHY0SmFxdGpFRWsxMEZLMHRMSTBST2RzTkZzYlcwZFJHNkhJ?=
 =?utf-8?B?TUVQL04vd3VTYzJXNFV4MXNib3c0SzJVNGxzYXByMVE2L0NqcjFjWS9uTjdC?=
 =?utf-8?B?VmlVWE5IWnFmeHdOYm43ckErWVg1UmRqMnBzalBLN3pWRVVkdzFGL1JsTWNq?=
 =?utf-8?B?WkVjYWhsOXZIRTE2UUVQYXcvdCtXT2s0V1RFbDh3TWc1ZXFlTFN0N2ZneGV5?=
 =?utf-8?B?QW4xd0dZMHRZRnVneDB0Zm1kQk9VTDdGVnl0MUJYUFpEYThNdHMyVGRIUGlL?=
 =?utf-8?B?bzdNQ285TGYyc2s3NnE5MHVDOEJiWlh2SlVud1Y0c0xrV2tPUSsxR1VZdFRm?=
 =?utf-8?B?T2M1bUhaRExHVlZUeVlCK2p5SmlqN1RvQi9XQjFVazlvMWpkTnFlaXBnL29E?=
 =?utf-8?B?TVBkUkdMSVpVc0JDb2xDVEw2anAxV0J5aVJINU1vSTJKdGMranhjZmJGcjBO?=
 =?utf-8?B?R3BQUlZnTi93T05lcXYwWUJHcHhXWkVaaDluNzV1dFlUa3BuWnN4Zk02N3Bx?=
 =?utf-8?B?eGlPUjROLzI0OWYwWFBoYWJpOWtPZXQ1WW1ScDVBNkdmamxIWGNRc0JoeFR3?=
 =?utf-8?B?MWVhU00wOWFOWlQyZFFNa2JuaFVhM2gyN25teDZWNFd4NHE3bzdZM0NNRWRr?=
 =?utf-8?B?ZnpuUWgwSU1qSVM0UDQrMWorUk93SnRmUzArbkw5dWNjQ1VRNmR1TXRWZVIr?=
 =?utf-8?B?MUQ0aGNBNjVRTWRHSHZJZ1p3VDBDaWlhSFM0SFkrSXV2SmFXeDdrTkZDdlU1?=
 =?utf-8?B?bUJoTWRORGJmaE9SUTJLRGNXdkpMRzRZQmN6eWs2QzNjZnUxbENPNFBJOWds?=
 =?utf-8?B?QVh2bEtSUU54cks1WWtCbU5sbjZOKzFydFJ0Qi9LbGhMRjN5Si90SE1nMWxh?=
 =?utf-8?B?MjBvamcvWGVtK2JBRmJiR2NlSy82RkhiT1c1NDRIOEhscjJzOTJkWVMrTnE4?=
 =?utf-8?B?MDJrVTFLaURCTERsUkxVeG5MSUg2L0xvZHQ2N3hvR0Rra3ZVdjE4R1M4alVW?=
 =?utf-8?B?aUY1dmZGTVdjRi9PajVlRE4wQ3l0L1lKSUZmSjRtVzllT2l3YTRsWWc3MEhh?=
 =?utf-8?B?cGFLZXJOVFA5TUtSOTloR3VrdEtBYW9hYXB4eHp5cFhCRXlqelZnRmFlaHlh?=
 =?utf-8?B?Qm1qQ3NXUnUzYmphWDBmdDRjanpYYjFZQ3dYM1FqMzh6TFhVRFZmbHdjWkE3?=
 =?utf-8?B?bXBZQ1poVkpYMHhSLzZHWjlRZ01GZ2xDS1RRMlRqcDJURlBkQ21kck5Handl?=
 =?utf-8?B?Y1diRnRrZnQzS3pGY0xZWlZFNi96bTNIMXlZZ0RONUtHY2hYaE56cS9GT002?=
 =?utf-8?B?d0tIT3k3Nk50ajJhY2puYk81bjdVdGpYWmVlTmM1UXEyOEVzNCs0b0JFbERE?=
 =?utf-8?B?YlE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 7d5f3b5c-cf6c-4046-a2b8-08da9021feab
X-MS-Exchange-CrossTenant-AuthSource: DM4PR11MB5373.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Sep 2022 16:08:10.8734
 (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: DTJdjz5fNfhXXWg20OQn4cmMEmgTQ1qmc/twScgKnQ8BZeHq8V1qiPNnY3MsLhzp9WqRh0Svtx/osBpJJfGO+INp/V6LNQ7mFEmfsIO6SLA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR11MB1820
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI,
        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

On Tue, Sep 06, 2022 at 03:21:25PM +0100, Matthew Wilcox wrote:
> On Tue, Sep 06, 2022 at 04:01:13PM +0200, Michał Winiarski wrote:
> > 64 DRM device nodes is not enough for everyone.
> > Upgrade it to ~512K (which definitely is more than enough).
> > 
> > To allow testing userspace support for >64 devices, add additional DRM
> > modparam (skip_legacy_minors) which causes DRM to skip allocating minors
> > in 0-192 range.
> > Additionally - one minor tweak around minor DRM IDR locking and IDR lockdep
> > annotations.
> 
> The IDR is deprecated; rather than making all these changes around
> the IDR, could you convert it to use the XArray instead?  I did it
> once before, but those patches bounced off the submissions process.

Sure. The IDR annotation can still be useful for existing users though,
are you saying I should drop it as well?

-Michał