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: <77cee477-6b72-3eb1-487b-33e12d7418ac@synaptics.com>
Date:   Tue, 29 Nov 2022 18:46:03 +0800
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.0
Subject: Re: [RFC] drm/fourcc: Add a modifier for contiguous memory
Content-Language: en-US
To:     Simon Ser <contact@emersion.fr>
Cc:     dri-devel@lists.freedesktop.org, ayaka@soulik.info,
        linux-kernel@vger.kernel.org, nicolas@ndufresne.ca,
        hverkuil@xs4all.nl, tzimmermann@suse.de, tfiga@chromium.org,
        linux-media@vger.kernel.org
References: <20221129101030.57499-1-randy.li@synaptics.com>
 <sHR1F4GVsLkT7F5iPXtRNZMZ1R_vSHulae_ZqcpUXP6_u3t85f_JnERnKg5Pombd1gDlNi24rBVbeDGK6XddDTzyRPbipWsmRt78DDDl4o0=@emersion.fr>
From:   Hsia-Jun Li <Randy.Li@synaptics.com>
In-Reply-To: <sHR1F4GVsLkT7F5iPXtRNZMZ1R_vSHulae_ZqcpUXP6_u3t85f_JnERnKg5Pombd1gDlNi24rBVbeDGK6XddDTzyRPbipWsmRt78DDDl4o0=@emersion.fr>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?TTRWYmlVS0kyZ21USzJDWllTN0U3TnZabWZhNmhBRFptVUd1cmZRaVFRWEhz?=
 =?utf-8?B?SGxKaGo0eWFHZHlUWURINkNpR1R6V1NydG1HU3Fka0ljWFFBWG5od1FQcUw4?=
 =?utf-8?B?Sm9weDlKZm1idkEvTE5DRlArN2hFZkRFYzc4NW51Vlc0dUQyWWNjWmtkOW9o?=
 =?utf-8?B?OE9Yb1U0WVdLdWtycHNCd3FlbkxKWFY5c1A1UFE2Z1NmUkxNcGQvWUNmUVBZ?=
 =?utf-8?B?b3VFSDdtbThMVFNpU0ExZmNjTkI2d0RhZFVRenBselRYOU1DbzhHcE5FUmNP?=
 =?utf-8?B?MGdwYUdibUV2U3hDT3dSR3hqSmc4V0hQdW1na3pSaXMzdXF6ZHgvbnJucnpv?=
 =?utf-8?B?QkFROStGWGhsbG52bUgwT3daY3NwRUZlbEU3VzVKQURKMVRxRkxoM1gzbWxr?=
 =?utf-8?B?NmxTQ0dZS2hoeWtiOUV1VmdFclRKazhCZkNtMHVCV2lpYUxtZHRkV1I2NGk3?=
 =?utf-8?B?QVFjcmh1Unc0bUlNdHJnMko1UkE2aWhCZTdwUnh1RWo0RjcyS0NRdmN3UVlT?=
 =?utf-8?B?K29Vd3RvZ3BaaFNtSnhqeUlOcTdHeEhrVGFNVEswNHQwenNvaXlSUytDNFJq?=
 =?utf-8?B?SHFvRVpaNElOckN2VzA1NTVYUk10cDNNeC9ja1hQZGtRT0ZjNEtXTmFsYlRa?=
 =?utf-8?B?aFl5WEQrUjNZMXpFdWU4WWtCZ1VzWXJtTEJ3bjFZc0gxcjVGUEc5QmtUdTRn?=
 =?utf-8?B?TGhiMExDQWJxT3BnS25iTXNXSHh6VkRzTnlML2o0c1BkMXNaUU9BMlo0TmtK?=
 =?utf-8?B?RjJ1YXVWUmFBWEtDelBQeDZISXp2K1F5OGREWVpONlZTbTFLRHJmSy9vZ1FY?=
 =?utf-8?B?RkVXUHdKWGYzZjllMStoM2RKM3dpS1JVaDFCRDAyTitDc094WFlZQm1lYTZR?=
 =?utf-8?B?aEppZUlCQ1ZrOENjVTVscjJ0N0greURKczNuYk5pdVB6WlphaGNRcFV0cytD?=
 =?utf-8?B?TXJ4KzVTS0ZycGg4a3lXR0VNamhjQU0wWWxrbEJreXVJSkxvYnhTZDc0UHk1?=
 =?utf-8?B?aWlBSkp5R3ZUMWt1T1N5Ry9qQWdUZ0lHU0VIeUZHK3hJV1A4b0k4ZUJrYmw4?=
 =?utf-8?B?c3AzTms4UFZwREFuUTN1TG4xZ3FaYTdRbmUrMGl2Ly92UFJ6KzJPNVR3VXBU?=
 =?utf-8?B?NFVyMThzQlU0V2MxcWtNZGRaMisyZ21NQ3RyblB5OHp5eWVVaEpGQlFQUmxQ?=
 =?utf-8?B?R0JiRERNRkJlcVV2KzVMcC9uaTMxN0NUc0Y3bVNOMStFNWRzMnlQc3VnT09z?=
 =?utf-8?B?R1RqSEt1bnNtYzlrd1huWmpmT0VneHdnUWhoUTJvNDlPMGtMdmhpbytWMWRV?=
 =?utf-8?B?cmN3WGdjRUZSaVZoNlAyMmJkSWl2OE5PQlhPbjc4akxVcUpOYnVlVk5rVDJm?=
 =?utf-8?B?Zll3RGdBVEFRbE5saTViOGdNZ3RoTnd3RWpCYjcxQlgxSnRZMm5uSTduNjFP?=
 =?utf-8?B?bXhZNEFVaWF3WjhmVitLd1EzWjBwZGgrODJqeVVjYkVkZk1YeDZIKzJ4c0pX?=
 =?utf-8?B?S1pUWk5lSkY3WjBVQmJOZnlqU2ZIU3UzbWdHQysyT2VvVFFqVGZrTDU2bEk4?=
 =?utf-8?B?SUhnSGo3VlBSYkF5YjhoUGpxWFhmRkV2bmdPdVNXcTVyZ3ZnRFNON0FsaXA3?=
 =?utf-8?B?NFNYcmNtR2VwK29wOFcxS2JidWh6WmVjZy9UN3l4N3hRM0JYVDJ3eW9wd2tl?=
 =?utf-8?B?bWwrcUtBVXBYalpBZnBqMGR4SVpNWWh0ZWRNMGFHQW85Z0JCVmlRWGpJdWdD?=
 =?utf-8?B?MlZ2RnhyYlhiYTdlUkd0ck9CbjN5Y3J6TG1pTldDRmtrTnNBZFcyYmVKNWZU?=
 =?utf-8?B?aUkyTStMckFwKzFjaVlPeHpsYWpUeTVERi9WZDM3Q01acG9MR0ZRanY1dEgx?=
 =?utf-8?B?Y0NIRDNTQjdidDVsbnhkeEsvZmc2aE5OaHVXUWhoTDJVV0hBRlFIdHFaMGJW?=
 =?utf-8?B?MkhMTUg0SXBmV1hNL1MvckNCVGtaOGh2WThIRUVQOUdBT0VDc2w0MkxZTzdi?=
 =?utf-8?B?eGNZclpFY0d6OVhLVGZaeE9YNy9Wd1JmZmpWY1FoQ1U0dEo5WUNlT0NOWWFS?=
 =?utf-8?B?WjhiUUhxeEZCWFNvN3hPMUFwTm9aRkNicDU4MTNVbnE2SENyc2ZmRnhOK1Zp?=
 =?utf-8?Q?PfQNDe3++xmfyX3q2+QBlhYwb?=
X-OriginatorOrg: synaptics.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d7038b88-0685-4e70-1a36-08dad1f6ed2c
X-MS-Exchange-CrossTenant-AuthSource: DM6PR03MB5196.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Nov 2022 10:46:09.8690
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 335d1fbc-2124-4173-9863-17e7051a2a0e
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: nxLBCXX50cVnw8/+ZcsfHuhkGL41OHydJVr595xSdeCfZS6+6d23iDSOq1hvrZygQ7lKJGCPwib5JN8E1o+klg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR03MB5615
X-Spam-Status: No, score=-2.2 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,
        SPF_HELO_PASS,SPF_PASS 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 11/29/22 18:18, Simon Ser wrote:
> CAUTION: Email originated externally, do not click links or open attachments unless you recognize the sender and know the content is safe.
> 
> 
> Format modifiers are for the buffer layout only, not for the allocation
> parameters, placement, etc. See the doc comment at the top of
> drm_fourcc.h.
In the v4l2 mail list, we have such proposal that dropping the pixel 
formats(not the codec formats) from v4l2 header completely, as the 
growing of tile pixel formats.
But we can't get rid of those variants about non-contiguous(the same 
value FOURCC in v4l2 are all for the contiguous memory).

Before I solve this problem, I believe the support for tile formats in 
v4l2 would never be stable.

The most common way here is to hack the pixel format modifier, then 
userspace library could be aware this in allocation and get properties 
of the drm_planes.

Or another way, we could add a common plane property, indicated that 
whether the driver requests contiguous memory plane for a format?
-- 
Hsia-Jun(Randy) Li