Re: [PATCH 0/3] Tools: hv: vssdaemon: freeze/thaw logic improvement for the failure case

2014-11-10 Thread Vitaly Kuznetsov
Dexuan Cui  writes:

>> -Original Message-
>> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com]
>> Sent: Saturday, November 8, 2014 1:09 AM
>> To: KY Srinivasan; Haiyang Zhang; Greg Kroah-Hartman
>> Cc: de...@linuxdriverproject.org; linux-kernel@vger.kernel.org; Dexuan Cui
>> Subject: [PATCH 0/3] Tools: hv: vssdaemon: freeze/thaw logic improvement
>> for the failure case
>> 
>> This patch series addresses the following issues:
>> - Wrong error reporting for multiple filesystems case.
>> - Skip all readonly-mounted filesystems instead of skipping iso9660.
>> - Thaw all filesystems after an unsuccessful freeze attempt.
>> 
>> Vitaly Kuznetsov (3):
>>   Tools: hv: vssdaemon: consult with errno in case of failure only
>>   Tools: hv: vssdaemon: skip all filesystems mounted readonly
>>   Tools: hv: vssdaemon: thaw everything in case of freeze failure
>> 
>>  tools/hv/hv_vss_daemon.c | 14 --
>>  1 file changed, 12 insertions(+), 2 deletions(-)
>
> Hi Vitaly,
> Thanks for your patchset!
>
> FYI: Greg checked in a patch of mine several hours ago -- my patch
> implemented "thaw all filesytems on a failure of freeze" too. :-)

Ah, sorry for stepping on your toes :-)

>
> Please see my patch in Greg's char-misc-next tree:
> https://git.kernel.org/cgit/linux/kernel/git/gregkh/char-misc.git/commit/?h=char-misc-next=4f689190bb55d171d2f6614f8a6cbd4b868e48bd
>
> Can you please rebase  your patch(es) on Greg's tree?

Sure, I'll throw away my patch#3, rebase, and repost.

>
> Thanks,
> -- Dexuan

-- 
  Vitaly
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/3] Tools: hv: vssdaemon: freeze/thaw logic improvement for the failure case

2014-11-10 Thread Vitaly Kuznetsov
Dexuan Cui de...@microsoft.com writes:

 -Original Message-
 From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com]
 Sent: Saturday, November 8, 2014 1:09 AM
 To: KY Srinivasan; Haiyang Zhang; Greg Kroah-Hartman
 Cc: de...@linuxdriverproject.org; linux-kernel@vger.kernel.org; Dexuan Cui
 Subject: [PATCH 0/3] Tools: hv: vssdaemon: freeze/thaw logic improvement
 for the failure case
 
 This patch series addresses the following issues:
 - Wrong error reporting for multiple filesystems case.
 - Skip all readonly-mounted filesystems instead of skipping iso9660.
 - Thaw all filesystems after an unsuccessful freeze attempt.
 
 Vitaly Kuznetsov (3):
   Tools: hv: vssdaemon: consult with errno in case of failure only
   Tools: hv: vssdaemon: skip all filesystems mounted readonly
   Tools: hv: vssdaemon: thaw everything in case of freeze failure
 
  tools/hv/hv_vss_daemon.c | 14 --
  1 file changed, 12 insertions(+), 2 deletions(-)

 Hi Vitaly,
 Thanks for your patchset!

 FYI: Greg checked in a patch of mine several hours ago -- my patch
 implemented thaw all filesytems on a failure of freeze too. :-)

Ah, sorry for stepping on your toes :-)


 Please see my patch in Greg's char-misc-next tree:
 https://git.kernel.org/cgit/linux/kernel/git/gregkh/char-misc.git/commit/?h=char-misc-nextid=4f689190bb55d171d2f6614f8a6cbd4b868e48bd

 Can you please rebase  your patch(es) on Greg's tree?

Sure, I'll throw away my patch#3, rebase, and repost.


 Thanks,
 -- Dexuan

-- 
  Vitaly
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH 0/3] Tools: hv: vssdaemon: freeze/thaw logic improvement for the failure case

2014-11-07 Thread Dexuan Cui
> -Original Message-
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com]
> Sent: Saturday, November 8, 2014 1:09 AM
> To: KY Srinivasan; Haiyang Zhang; Greg Kroah-Hartman
> Cc: de...@linuxdriverproject.org; linux-kernel@vger.kernel.org; Dexuan Cui
> Subject: [PATCH 0/3] Tools: hv: vssdaemon: freeze/thaw logic improvement
> for the failure case
> 
> This patch series addresses the following issues:
> - Wrong error reporting for multiple filesystems case.
> - Skip all readonly-mounted filesystems instead of skipping iso9660.
> - Thaw all filesystems after an unsuccessful freeze attempt.
> 
> Vitaly Kuznetsov (3):
>   Tools: hv: vssdaemon: consult with errno in case of failure only
>   Tools: hv: vssdaemon: skip all filesystems mounted readonly
>   Tools: hv: vssdaemon: thaw everything in case of freeze failure
> 
>  tools/hv/hv_vss_daemon.c | 14 --
>  1 file changed, 12 insertions(+), 2 deletions(-)

Hi Vitaly,
Thanks for your patchset!

FYI: Greg checked in a patch of mine several hours ago -- my patch
implemented "thaw all filesytems on a failure of freeze" too. :-)

Please see my patch in Greg's char-misc-next tree:
https://git.kernel.org/cgit/linux/kernel/git/gregkh/char-misc.git/commit/?h=char-misc-next=4f689190bb55d171d2f6614f8a6cbd4b868e48bd

Can you please rebase  your patch(es) on Greg's tree?

Thanks,
-- Dexuan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH 0/3] Tools: hv: vssdaemon: freeze/thaw logic improvement for the failure case

2014-11-07 Thread Dexuan Cui
 -Original Message-
 From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com]
 Sent: Saturday, November 8, 2014 1:09 AM
 To: KY Srinivasan; Haiyang Zhang; Greg Kroah-Hartman
 Cc: de...@linuxdriverproject.org; linux-kernel@vger.kernel.org; Dexuan Cui
 Subject: [PATCH 0/3] Tools: hv: vssdaemon: freeze/thaw logic improvement
 for the failure case
 
 This patch series addresses the following issues:
 - Wrong error reporting for multiple filesystems case.
 - Skip all readonly-mounted filesystems instead of skipping iso9660.
 - Thaw all filesystems after an unsuccessful freeze attempt.
 
 Vitaly Kuznetsov (3):
   Tools: hv: vssdaemon: consult with errno in case of failure only
   Tools: hv: vssdaemon: skip all filesystems mounted readonly
   Tools: hv: vssdaemon: thaw everything in case of freeze failure
 
  tools/hv/hv_vss_daemon.c | 14 --
  1 file changed, 12 insertions(+), 2 deletions(-)

Hi Vitaly,
Thanks for your patchset!

FYI: Greg checked in a patch of mine several hours ago -- my patch
implemented thaw all filesytems on a failure of freeze too. :-)

Please see my patch in Greg's char-misc-next tree:
https://git.kernel.org/cgit/linux/kernel/git/gregkh/char-misc.git/commit/?h=char-misc-nextid=4f689190bb55d171d2f6614f8a6cbd4b868e48bd

Can you please rebase  your patch(es) on Greg's tree?

Thanks,
-- Dexuan
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/