* [PATCH] btrfs-progs: Add mount point output for 'btrfs fi df' command.
@ 2014-07-08 5:43 Qu Wenruo
2014-07-10 0:41 ` Satoru Takeuchi
0 siblings, 1 reply; 6+ messages in thread
From: Qu Wenruo @ 2014-07-08 5:43 UTC (permalink / raw)
To: linux-btrfs; +Cc: dsterba
Add mount point output for 'btrfs fi df'.
Also since the patch uses find_mount_root() to find mount point,
now 'btrfs fi df' can output more meaningful error message when given a
non-btrfs path.
Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
---
This patch needs to be merged after the following path:
btrfs-progs: Check fstype in find_mount_root()
---
cmds-filesystem.c | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/cmds-filesystem.c b/cmds-filesystem.c
index 4b2d27e..d571765 100644
--- a/cmds-filesystem.c
+++ b/cmds-filesystem.c
@@ -187,12 +187,22 @@ static int cmd_filesystem_df(int argc, char **argv)
int ret;
int fd;
char *path;
+ char *mount_point = NULL;
DIR *dirstream = NULL;
if (check_argc_exact(argc, 2))
usage(cmd_filesystem_df_usage);
path = argv[1];
+ ret = find_mount_root(path, &mount_point);
+ if (ret < 0) {
+ if (ret != -ENOENT)
+ fprintf(stderr, "ERROR: Failed to find mount root for path %s: %s\n",
+ path, strerror(-ret));
+ return 1;
+ }
+ printf("Mounted on: %s\n", mount_point);
+ free(mount_point);
fd = open_file_or_dir(path, &dirstream);
if (fd < 0) {
--
2.0.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] btrfs-progs: Add mount point output for 'btrfs fi df' command.
2014-07-08 5:43 [PATCH] btrfs-progs: Add mount point output for 'btrfs fi df' command Qu Wenruo
@ 2014-07-10 0:41 ` Satoru Takeuchi
2014-07-10 0:59 ` Qu Wenruo
0 siblings, 1 reply; 6+ messages in thread
From: Satoru Takeuchi @ 2014-07-10 0:41 UTC (permalink / raw)
To: Qu Wenruo, linux-btrfs; +Cc: dsterba
Hi Qu,
(2014/07/08 14:43), Qu Wenruo wrote:
> Add mount point output for 'btrfs fi df'.
> Also since the patch uses find_mount_root() to find mount point,
> now 'btrfs fi df' can output more meaningful error message when given a
> non-btrfs path.
>
> Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
> ---
> This patch needs to be merged after the following path:
> btrfs-progs: Check fstype in find_mount_root()
> ---
> cmds-filesystem.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/cmds-filesystem.c b/cmds-filesystem.c
> index 4b2d27e..d571765 100644
> --- a/cmds-filesystem.c
> +++ b/cmds-filesystem.c
> @@ -187,12 +187,22 @@ static int cmd_filesystem_df(int argc, char **argv)
> int ret;
> int fd;
> char *path;
> + char *mount_point = NULL;
> DIR *dirstream = NULL;
>
> if (check_argc_exact(argc, 2))
> usage(cmd_filesystem_df_usage);
>
> path = argv[1];
> + ret = find_mount_root(path, &mount_point);
> + if (ret < 0) {
> + if (ret != -ENOENT)
Is "if (ret != -ENOENT)" to avoid the error message duplication
with the following code?
utils.c:
===============================================================================
...
int find_mount_root(...)
{
...
if (!longest_match) {
fprintf(stderr,
"ERROR: Failed to find mount root for path %s.\n",
path);
return -ENOENT;
}
...
===============================================================================
I consider making the following two patches is the better way.
- Patch 1. Removing this error message from find_mount_root().
This cause no problem since all the current find_mount_root()
caller show their own error message.
- Patch 2. Your patch with removing that if sentence.
Thanks,
Satoru
> + fprintf(stderr, "ERROR: Failed to find mount root for path %s: %s\n",
> + path, strerror(-ret));
> + return 1;
> + }
> + printf("Mounted on: %s\n", mount_point);
> + free(mount_point);
>
> fd = open_file_or_dir(path, &dirstream);
> if (fd < 0) {
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] btrfs-progs: Add mount point output for 'btrfs fi df' command.
2014-07-10 0:41 ` Satoru Takeuchi
@ 2014-07-10 0:59 ` Qu Wenruo
2014-07-10 1:17 ` Qu Wenruo
2014-07-10 1:18 ` Qu Wenruo
0 siblings, 2 replies; 6+ messages in thread
From: Qu Wenruo @ 2014-07-10 0:59 UTC (permalink / raw)
To: Satoru Takeuchi, linux-btrfs; +Cc: dsterba
-------- Original Message --------
Subject: Re: [PATCH] btrfs-progs: Add mount point output for 'btrfs fi
df' command.
From: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>
To: Qu Wenruo <quwenruo@cn.fujitsu.com>, linux-btrfs@vger.kernel.org
Date: 2014年07月10日 08:41
> Hi Qu,
>
> (2014/07/08 14:43), Qu Wenruo wrote:
>> Add mount point output for 'btrfs fi df'.
>> Also since the patch uses find_mount_root() to find mount point,
>> now 'btrfs fi df' can output more meaningful error message when given a
>> non-btrfs path.
>>
>> Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
>> ---
>> This patch needs to be merged after the following path:
>> btrfs-progs: Check fstype in find_mount_root()
>> ---
>> cmds-filesystem.c | 10 ++++++++++
>> 1 file changed, 10 insertions(+)
>>
>> diff --git a/cmds-filesystem.c b/cmds-filesystem.c
>> index 4b2d27e..d571765 100644
>> --- a/cmds-filesystem.c
>> +++ b/cmds-filesystem.c
>> @@ -187,12 +187,22 @@ static int cmd_filesystem_df(int argc, char **argv)
>> int ret;
>> int fd;
>> char *path;
>> + char *mount_point = NULL;
>> DIR *dirstream = NULL;
>>
>> if (check_argc_exact(argc, 2))
>> usage(cmd_filesystem_df_usage);
>>
>> path = argv[1];
>> + ret = find_mount_root(path, &mount_point);
>> + if (ret < 0) {
>> + if (ret != -ENOENT)
> Is "if (ret != -ENOENT)" to avoid the error message duplication
> with the following code?
Yes.
>
> utils.c:
> ===============================================================================
> ...
> int find_mount_root(...)
> {
> ...
> if (!longest_match) {
> fprintf(stderr,
> "ERROR: Failed to find mount root for path %s.\n",
> path);
> return -ENOENT;
> }
> ...
> ===============================================================================
>
> I consider making the following two patches is the better way.
>
> - Patch 1. Removing this error message from find_mount_root().
> This cause no problem since all the current find_mount_root()
> caller show their own error message.
>
> - Patch 2. Your patch with removing that if sentence.
Thanks for the suggestion, it really makes sense.
I'll send the new version with other small modification like integrate
realpath() into find_mount_root()
since every caller of find_mount_root() does the realpath resolve.
Thanks,
Qu
>
> Thanks,
> Satoru
>
>> + fprintf(stderr, "ERROR: Failed to find mount root for path %s: %s\n",
>> + path, strerror(-ret));
>> + return 1;
>> + }
>> + printf("Mounted on: %s\n", mount_point);
>> + free(mount_point);
>>
>> fd = open_file_or_dir(path, &dirstream);
>> if (fd < 0) {
>>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] btrfs-progs: Add mount point output for 'btrfs fi df' command.
2014-07-10 0:59 ` Qu Wenruo
@ 2014-07-10 1:17 ` Qu Wenruo
2014-07-10 1:18 ` Qu Wenruo
1 sibling, 0 replies; 6+ messages in thread
From: Qu Wenruo @ 2014-07-10 1:17 UTC (permalink / raw)
To: Satoru Takeuchi, linux-btrfs; +Cc: dsterba
-------- Original Message --------
Subject: Re: [PATCH] btrfs-progs: Add mount point output for 'btrfs fi
df' command.
From: Qu Wenruo <quwenruo@cn.fujitsu.com>
To: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>,
linux-btrfs@vger.kernel.org
Date: 2014年07月10日 08:59
>
> -------- Original Message --------
> Subject: Re: [PATCH] btrfs-progs: Add mount point output for 'btrfs fi
> df' command.
> From: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>
> To: Qu Wenruo <quwenruo@cn.fujitsu.com>, linux-btrfs@vger.kernel.org
> Date: 2014年07月10日 08:41
>> Hi Qu,
>>
>> (2014/07/08 14:43), Qu Wenruo wrote:
>>> Add mount point output for 'btrfs fi df'.
>>> Also since the patch uses find_mount_root() to find mount point,
>>> now 'btrfs fi df' can output more meaningful error message when given a
>>> non-btrfs path.
>>>
>>> Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
>>> ---
>>> This patch needs to be merged after the following path:
>>> btrfs-progs: Check fstype in find_mount_root()
>>> ---
>>> cmds-filesystem.c | 10 ++++++++++
>>> 1 file changed, 10 insertions(+)
>>>
>>> diff --git a/cmds-filesystem.c b/cmds-filesystem.c
>>> index 4b2d27e..d571765 100644
>>> --- a/cmds-filesystem.c
>>> +++ b/cmds-filesystem.c
>>> @@ -187,12 +187,22 @@ static int cmd_filesystem_df(int argc, char
>>> **argv)
>>> int ret;
>>> int fd;
>>> char *path;
>>> + char *mount_point = NULL;
>>> DIR *dirstream = NULL;
>>> if (check_argc_exact(argc, 2))
>>> usage(cmd_filesystem_df_usage);
>>> path = argv[1];
>>> + ret = find_mount_root(path, &mount_point);
>>> + if (ret < 0) {
>>> + if (ret != -ENOENT)
>> Is "if (ret != -ENOENT)" to avoid the error message duplication
>> with the following code?
> Yes.
>>
>> utils.c:
>> ===============================================================================
>>
>> ...
>> int find_mount_root(...)
>> {
>> ...
>> if (!longest_match) {
>> fprintf(stderr,
>> "ERROR: Failed to find mount root for path
>> %s.\n",
>> path);
>> return -ENOENT;
>> }
>> ...
>> ===============================================================================
>>
>>
>> I consider making the following two patches is the better way.
>>
>> - Patch 1. Removing this error message from find_mount_root().
>> This cause no problem since all the current
>> find_mount_root()
>> caller show their own error message.
>> - Patch 2. Your patch with removing that if sentence.
> Thanks for the suggestion, it really makes sense.
> I'll send the new version with other small modification like integrate
> realpath() into find_mount_root()
> since every caller of find_mount_root() does the realpath resolve.
>
> Thanks,
> Qu
P.S. I prefer to integrate all fprintf() into find_mount_root(), since
that will make the error message more specific
and more easy to understandable.
Thanks,
Qu
>>
>> Thanks,
>> Satoru
>>
>>> + fprintf(stderr, "ERROR: Failed to find mount root
>>> for path %s: %s\n",
>>> + path, strerror(-ret));
>>> + return 1;
>>> + }
>>> + printf("Mounted on: %s\n", mount_point);
>>> + free(mount_point);
>>> fd = open_file_or_dir(path, &dirstream);
>>> if (fd < 0) {
>>>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] btrfs-progs: Add mount point output for 'btrfs fi df' command.
2014-07-10 0:59 ` Qu Wenruo
2014-07-10 1:17 ` Qu Wenruo
@ 2014-07-10 1:18 ` Qu Wenruo
2014-07-10 1:22 ` Satoru Takeuchi
1 sibling, 1 reply; 6+ messages in thread
From: Qu Wenruo @ 2014-07-10 1:18 UTC (permalink / raw)
To: Satoru Takeuchi, linux-btrfs; +Cc: dsterba
-------- Original Message --------
Subject: Re: [PATCH] btrfs-progs: Add mount point output for 'btrfs fi
df' command.
From: Qu Wenruo <quwenruo@cn.fujitsu.com>
To: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>,
linux-btrfs@vger.kernel.org
Date: 2014年07月10日 08:59
>
> -------- Original Message --------
> Subject: Re: [PATCH] btrfs-progs: Add mount point output for 'btrfs fi
> df' command.
> From: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>
> To: Qu Wenruo <quwenruo@cn.fujitsu.com>, linux-btrfs@vger.kernel.org
> Date: 2014年07月10日 08:41
>> Hi Qu,
>>
>> (2014/07/08 14:43), Qu Wenruo wrote:
>>> Add mount point output for 'btrfs fi df'.
>>> Also since the patch uses find_mount_root() to find mount point,
>>> now 'btrfs fi df' can output more meaningful error message when given a
>>> non-btrfs path.
>>>
>>> Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
>>> ---
>>> This patch needs to be merged after the following path:
>>> btrfs-progs: Check fstype in find_mount_root()
>>> ---
>>> cmds-filesystem.c | 10 ++++++++++
>>> 1 file changed, 10 insertions(+)
>>>
>>> diff --git a/cmds-filesystem.c b/cmds-filesystem.c
>>> index 4b2d27e..d571765 100644
>>> --- a/cmds-filesystem.c
>>> +++ b/cmds-filesystem.c
>>> @@ -187,12 +187,22 @@ static int cmd_filesystem_df(int argc, char
>>> **argv)
>>> int ret;
>>> int fd;
>>> char *path;
>>> + char *mount_point = NULL;
>>> DIR *dirstream = NULL;
>>> if (check_argc_exact(argc, 2))
>>> usage(cmd_filesystem_df_usage);
>>> path = argv[1];
>>> + ret = find_mount_root(path, &mount_point);
>>> + if (ret < 0) {
>>> + if (ret != -ENOENT)
>> Is "if (ret != -ENOENT)" to avoid the error message duplication
>> with the following code?
> Yes.
>>
>> utils.c:
>> ===============================================================================
>>
>> ...
>> int find_mount_root(...)
>> {
>> ...
>> if (!longest_match) {
>> fprintf(stderr,
>> "ERROR: Failed to find mount root for path
>> %s.\n",
>> path);
>> return -ENOENT;
>> }
>> ...
>> ===============================================================================
>>
>>
>> I consider making the following two patches is the better way.
>>
>> - Patch 1. Removing this error message from find_mount_root().
>> This cause no problem since all the current
>> find_mount_root()
>> caller show their own error message.
>> - Patch 2. Your patch with removing that if sentence.
> Thanks for the suggestion, it really makes sense.
> I'll send the new version with other small modification like integrate
> realpath() into find_mount_root()
> since every caller of find_mount_root() does the realpath resolve.
>
> Thanks,
> Qu
P.S. I prefer to integrate all fprintf() into find_mount_root(), since
that will make the error message more specific
and more easy to understandable.
So I'll remove the caller fprintf.
Would this be OK for you?
Thanks,
Qu
>>
>> Thanks,
>> Satoru
>>
>>> + fprintf(stderr, "ERROR: Failed to find mount root
>>> for path %s: %s\n",
>>> + path, strerror(-ret));
>>> + return 1;
>>> + }
>>> + printf("Mounted on: %s\n", mount_point);
>>> + free(mount_point);
>>> fd = open_file_or_dir(path, &dirstream);
>>> if (fd < 0) {
>>>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] btrfs-progs: Add mount point output for 'btrfs fi df' command.
2014-07-10 1:18 ` Qu Wenruo
@ 2014-07-10 1:22 ` Satoru Takeuchi
0 siblings, 0 replies; 6+ messages in thread
From: Satoru Takeuchi @ 2014-07-10 1:22 UTC (permalink / raw)
To: Qu Wenruo, linux-btrfs; +Cc: dsterba
(2014/07/10 10:18), Qu Wenruo wrote:
>
> -------- Original Message --------
> Subject: Re: [PATCH] btrfs-progs: Add mount point output for 'btrfs fi df' command.
> From: Qu Wenruo <quwenruo@cn.fujitsu.com>
> To: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>, linux-btrfs@vger.kernel.org
> Date: 2014年07月10日 08:59
>>
>> -------- Original Message --------
>> Subject: Re: [PATCH] btrfs-progs: Add mount point output for 'btrfs fi df' command.
>> From: Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com>
>> To: Qu Wenruo <quwenruo@cn.fujitsu.com>, linux-btrfs@vger.kernel.org
>> Date: 2014年07月10日 08:41
>>> Hi Qu,
>>>
>>> (2014/07/08 14:43), Qu Wenruo wrote:
>>>> Add mount point output for 'btrfs fi df'.
>>>> Also since the patch uses find_mount_root() to find mount point,
>>>> now 'btrfs fi df' can output more meaningful error message when given a
>>>> non-btrfs path.
>>>>
>>>> Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
>>>> ---
>>>> This patch needs to be merged after the following path:
>>>> btrfs-progs: Check fstype in find_mount_root()
>>>> ---
>>>> cmds-filesystem.c | 10 ++++++++++
>>>> 1 file changed, 10 insertions(+)
>>>>
>>>> diff --git a/cmds-filesystem.c b/cmds-filesystem.c
>>>> index 4b2d27e..d571765 100644
>>>> --- a/cmds-filesystem.c
>>>> +++ b/cmds-filesystem.c
>>>> @@ -187,12 +187,22 @@ static int cmd_filesystem_df(int argc, char **argv)
>>>> int ret;
>>>> int fd;
>>>> char *path;
>>>> + char *mount_point = NULL;
>>>> DIR *dirstream = NULL;
>>>> if (check_argc_exact(argc, 2))
>>>> usage(cmd_filesystem_df_usage);
>>>> path = argv[1];
>>>> + ret = find_mount_root(path, &mount_point);
>>>> + if (ret < 0) {
>>>> + if (ret != -ENOENT)
>>> Is "if (ret != -ENOENT)" to avoid the error message duplication
>>> with the following code?
>> Yes.
>>>
>>> utils.c:
>>> ===============================================================================
>>> ...
>>> int find_mount_root(...)
>>> {
>>> ...
>>> if (!longest_match) {
>>> fprintf(stderr,
>>> "ERROR: Failed to find mount root for path %s.\n",
>>> path);
>>> return -ENOENT;
>>> }
>>> ...
>>> ===============================================================================
>>>
>>> I consider making the following two patches is the better way.
>>>
>>> - Patch 1. Removing this error message from find_mount_root().
>>> This cause no problem since all the current find_mount_root()
>>> caller show their own error message.
>>> - Patch 2. Your patch with removing that if sentence.
>> Thanks for the suggestion, it really makes sense.
>> I'll send the new version with other small modification like integrate realpath() into find_mount_root()
>> since every caller of find_mount_root() does the realpath resolve.
>>
>> Thanks,
>> Qu
> P.S. I prefer to integrate all fprintf() into find_mount_root(), since that will make the error message more specific
> and more easy to understandable.
> So I'll remove the caller fprintf.
> Would this be OK for you?
It's OK for me.
Satoru
>
> Thanks,
> Qu
>>>
>>> Thanks,
>>> Satoru
>>>
>>>> + fprintf(stderr, "ERROR: Failed to find mount root for path %s: %s\n",
>>>> + path, strerror(-ret));
>>>> + return 1;
>>>> + }
>>>> + printf("Mounted on: %s\n", mount_point);
>>>> + free(mount_point);
>>>> fd = open_file_or_dir(path, &dirstream);
>>>> if (fd < 0) {
>>>>
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-07-10 1:22 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-08 5:43 [PATCH] btrfs-progs: Add mount point output for 'btrfs fi df' command Qu Wenruo
2014-07-10 0:41 ` Satoru Takeuchi
2014-07-10 0:59 ` Qu Wenruo
2014-07-10 1:17 ` Qu Wenruo
2014-07-10 1:18 ` Qu Wenruo
2014-07-10 1:22 ` Satoru Takeuchi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).