[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] tools/xenstored: Wire properly the command line option -M/--path-max


  • To: Julien Grall <julien@xxxxxxx>
  • From: Luca Fancellu <luca.fancellu@xxxxxxx>
  • Date: Wed, 21 Apr 2021 15:10:34 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=B1bNpradCH3xPuDnthBmcOncFykB4SfgAlQ2JkiSna4=; b=YcSlc13fNbp4NzO8FePlVTRXOVWJxSLLOC/9z9StyGifckEIAZ9n8IrLbPmDSOH1dg6sG/sHBvm0XxI4nwxRusNOaHWEkE9DGftRlm9wXRZ/m8+3nQ+iGSIHHdWwY8XAnp7idHhtro3uo4WclAh9G7Iy9IH5MmHFuxl5R08MWWWq5sXntsz+SX5yFyfLlAdRnhRmy8nzxW+OyfqB9D+e81W/+0FJdS/F0A4yqvrlsFmajQ9TFm/KBUbANuyNW7lkXv6pVkhfFBc90Iowi90kd8B2C08mMMHy63D1d8ZDfOUzinpNphgbFxuZHtcCI11gA9wGGv6u8TI+JZlwMwA7qA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Yg1bo/T9oY7WpoaPMuiTsi94BuL+ZWY79jIDDnFW9x3z9yIsCj0oPL41Jtp1MJxUUjOGYqt7Z2rXHabogVa4WwHpLZfhkpnSTbQMyPshba7IWTLSogMht5kq4jFcEsuIhnOjWXiPpEawmkz8qZLLds+OqQTKeVGEhSop0Jyjhpioij5m8Y7rfZHXIYMkubn3n9qzizi/UONDQpzPZZX6LNHnaHJMzfu5FZZGi5SlbCRQ456Z+I8BZerUQcTOQM+wr9V/Gu/SxtZz7ORL94Qy1t65aDNEUSpzZ/+ABOLwtq5Ho4VqbIetU4W/VKlC+GzI1M6AqiPwW5vaujBbVSeF3g==
  • Authentication-results-original: xen.org; dkim=none (message not signed) header.d=none;xen.org; dmarc=none action=none header.from=arm.com;
  • Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx, Julien Grall <jgrall@xxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Juergen Gross <jgross@xxxxxxxx>
  • Delivery-date: Wed, 21 Apr 2021 14:10:57 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: xen.org; dkim=none (message not signed) header.d=none;xen.org; dmarc=none action=none header.from=arm.com;


> On 21 Apr 2021, at 15:02, Julien Grall <julien@xxxxxxx> wrote:
> 
> From: Julien Grall <jgrall@xxxxxxxxxx>
> 
> The command line option -M/--path-max was meant to be added by
> commit 924bf8c793cb "tools/xenstore: rework path length check" but this
> wasn't wired through properly.
> 
> Fix it by adding the missing "case 'M':".
> 
> Fixes: 924bf8c793cb ("tools/xenstore: rework path length check")
> Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>
> ---
> tools/xenstore/xenstored_core.c | 1 +
> 1 file changed, 1 insertion(+)
> 
> diff --git a/tools/xenstore/xenstored_core.c b/tools/xenstore/xenstored_core.c
> index 591b28e4552f..c638e46221eb 100644
> --- a/tools/xenstore/xenstored_core.c
> +++ b/tools/xenstore/xenstored_core.c
> @@ -2148,6 +2148,7 @@ int main(int argc, char *argv[])
>               case 'A':
>                       quota_nb_perms_per_node = strtol(optarg, NULL, 10);
>                       break;
> +             case 'M':
>                       quota_max_path_len = strtol(optarg, NULL, 10);
>                       quota_max_path_len = min(XENSTORE_REL_PATH_MAX,
>                                                quota_max_path_len);
>
> 2.17.1
> 
> 

Reviewed-by: Luca Fancellu <luca.fancellu@xxxxxxx>


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.