Re: MVFS results

2009-07-21 Thread Corinna Vinschen
On Jul 21 14:32, Eric Blake wrote: > Corinna Vinschen cygwin.com> writes: > > > How cute. I have reworked the patch slightly to encapsulate the MVFS > > code somewhat more. I've checked it in. Please try if it still works > > as advertised. > > Your patch swapped the order of the actions in r

Re: MVFS results

2009-07-21 Thread Eric Blake
Corinna Vinschen cygwin.com> writes: > How cute. I have reworked the patch slightly to encapsulate the MVFS > code somewhat more. I've checked it in. Please try if it still works > as advertised. Your patch swapped the order of the actions in relation to mine (by working on the original then

Re: MVFS results

2009-07-21 Thread Corinna Vinschen
On Jul 20 20:13, Eric Blake wrote: > SUCCESS! The following patch works for both read-only and writeable files. > Doing the work on the original handle is not enough - it has to be flushed to > disk so as not to be lost by later state changes. And doing the work on the > duplicate is not enou

Re: MVFS results

2009-07-20 Thread Eric Blake
Corinna Vinschen cygwin.com> writes: > > Or maybe this is a case where we need a FlushFileBuffer call during the > > utimens_fs after all, in addition to closing the duplicate descriptor, so that > > the original handle no longer has any state that needs flushing which might > > inadvertently

Re: MVFS results

2009-07-20 Thread Corinna Vinschen
On Jul 20 16:58, Eric Blake wrote: > Corinna Vinschen cygwin.com> writes: > > + InitializeObjectAttributes (&attr, &ro_u_empty, > > pc.objcaseinsensitive > (), > > + fh, NULL); > > Yet the obvious fix of using get_handle() instead of fh doesn't help either. O

Re: MVFS results

2009-07-20 Thread Eric Blake
Corinna Vinschen cygwin.com> writes: > Just to be sure. This does not occur with cp -p on other filesystems > than MVFS, right? I don't see that problem on NTFS or FAT. I can confirm that with both NTFS and FAT, the NtSetInformationFile in utimens_fs is immediate (no fd has to be closed). Th

Re: MVFS results

2009-07-20 Thread Corinna Vinschen
On Jul 20 15:02, Eric Blake wrote: > Corinna Vinschen cygwin.com> writes: > > > Found the root cause. R/O vs. writeable.> > > It's not, unfortunately. There is no code path in Cygwin's utimens > > and friends which temorarily resets the R/O attribute. If the R/O > > attribute is actually remov

Re: MVFS results

2009-07-20 Thread Eric Blake
Corinna Vinschen cygwin.com> writes: > > Found the root cause. R/O vs. writeable.> > It's not, unfortunately. There is no code path in Cygwin's utimens > and friends which temorarily resets the R/O attribute. If the R/O > attribute is actually removed temporarily, then touch calls chmod > or

Re: volinfo (was: MVFS results)

2009-07-20 Thread Marc Girod
nt... Marc -- View this message in context: http://www.nabble.com/MVFS-results-tp24504520p24567765.html Sent from the Cygwin list mailing list archive at Nabble.com. -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentation: http

Re: MVFS results

2009-07-18 Thread Corinna Vinschen
On Jul 17 23:15, Eric Blake wrote: > Eric Blake byu.net> writes: > > > > As I said above, I assume that MVFS has a problem similar to what we > > > have for remote HPFS. It's not sufficient to set only the minimal > > > set of access flags in calls to NtCreateFile/NtOpenFile in some > > > circum

Re: MVFS results

2009-07-17 Thread Eric Blake
Eric Blake byu.net> writes: > > As I said above, I assume that MVFS has a problem similar to what we > > have for remote HPFS. It's not sufficient to set only the minimal > > set of access flags in calls to NtCreateFile/NtOpenFile in some > > circumstances. I can't tell where the problem is for

Re: MVFS results

2009-07-17 Thread Corinna Vinschen
On Jul 16 17:15, Eric Blake wrote: > Corinna Vinschen cygwin.com> writes: > > Eric Blake wrote: > > > Maybe for MVFS it would be better to > > > return EBUSY instead of letting unlink succeed when the handle is > > > still open by another process: > > > > That would be something I could add, but

Re: MVFS results

2009-07-17 Thread Eric Blake
Corinna Vinschen cygwin.com> writes: > && !is_mvfs (RtlEqualUnicodePathPrefix (&fsname, &ro_u_mvfs, FALSE)) > > This checks for a MVFS prefix instead of equality of the entire string. Progess report. I can confirm that with cygwin1.dll built from the latest CVS, emacs can now create and des

Re: MVFS results

2009-07-17 Thread Steve Thompson
On Fri, 17 Jul 2009, Dave Korn wrote: I wanted to "save" one, I did that by leaving the power on indefinitely ... You had power? Luxury! I had to keep pedaling the bicycle! -s -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentatio

Re: MVFS results

2009-07-17 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Corinna Vinschen on 7/17/2009 1:15 AM: > Can we be sure that every version of MVFS has this obvious bug in > the name? Given that it's a bug in the first place, probably not. > I'd rather replace the following line in mount.cc, > fs_inf

Re: MVFS results

2009-07-17 Thread Corinna Vinschen
On Jul 17 01:17, Eric Blake wrote: > Eric Blake byu.net> writes: > > > Does the fact that MVFS is including the trailing NUL in fsname.Length, but > > ro_u_mvfs is not, relevant to RtlEqualUnicodeString returning false, even > > though the two Buffers are identical over their MaximumLength? > >

Re: MVFS results

2009-07-16 Thread Eric Blake
Eric Blake byu.net> writes: > Does the fact that MVFS is including the trailing NUL in fsname.Length, but > ro_u_mvfs is not, relevant to RtlEqualUnicodeString returning false, even > though the two Buffers are identical over their MaximumLength? > Sure enough, your patch plus this one-liner

Re: MVFS results

2009-07-16 Thread Dave Korn
Karl M wrote: >> Date: Thu, 16 Jul 2009 20:02:48 -0400 >> From: cgf >> Subject: Re: MVFS results >> >> >> I always just use the toggle switches on the front panel of my computer. >> >> Much easier. >> >> cgf >> > Do you prefer a

RE: MVFS results

2009-07-16 Thread Karl M
> Date: Thu, 16 Jul 2009 20:02:48 -0400 > From: cgf > Subject: Re: MVFS results > > > I always just use the toggle switches on the front panel of my computer. > > Much easier. > > cgf > Do you prefer a pin or a bent paperclip to poke new

Re: MVFS results

2009-07-16 Thread Christopher Faylor
On Thu, Jul 16, 2009 at 11:13:43PM +, Eric Blake wrote: >Corinna Vinschen cygwin.com> writes: >>Oh well. Ok, I planned to check this change in and let you then test >>from CVS. But since that's a problem for you, here's the patch. The >>important part is that Cygwin should now always create

Re: MVFS results

2009-07-16 Thread Eric Blake
Corinna Vinschen cygwin.com> writes: > Oh well. Ok, I planned to check this change in and let you then test > from CVS. But since that's a problem for you, here's the patch. The > important part is that Cygwin should now always create symlinks as > shortcuts on MVFS, regardless of the CYGWIN=w

Re: MVFS results

2009-07-16 Thread Corinna Vinschen
On Jul 16 17:15, Eric Blake wrote: > Corinna Vinschen cygwin.com> writes: > > Erm... it tries to use the recycle bin? Why? AFAICS, MVFS has the > > FILE_REMOTE_DEVICE device characteristic set, so it's logically always a > > remote drive and unlink_nt does not try to move the file to the recycle

Re: MVFS results

2009-07-16 Thread Eric Blake
Corinna Vinschen cygwin.com> writes: > > There are still issues trying to delete open files, where readdir() > > and stat() still see the file after it has been deleted, but utimes() > > and open() fail because it has been marked for deletion. Do you need > > an strace() of the rm and what it at

Re: MVFS results

2009-07-16 Thread Corinna Vinschen
On Jul 16 16:25, Eric Blake wrote: > Corinna Vinschen cygwin.com> writes: > > > > > For instance, maybe it chokes on the sharing flags. I'd try with > > > > FILE_SHARE_VALID_FLAGS instead of just FILE_SHARE_DELETE. > > Nope, still chokes on FILE_SHARE_VALID_FLAGS with access denied. Too bad.

Re: MVFS results

2009-07-16 Thread Eric Blake
Eric Blake byu.net> writes: > $ touch foo > touch: cannot touch `foo': Permission denied > $ exec 6 bash: bar: Permission denied I merged two different tests into one summary; this particular command should have read: $ exec 6 $ exec 5<&- At which point, trying again gives: $ exec 6http://cy

Re: MVFS results

2009-07-16 Thread Eric Blake
Corinna Vinschen cygwin.com> writes: > > > For instance, maybe it chokes on the sharing flags. I'd try with > > > FILE_SHARE_VALID_FLAGS instead of just FILE_SHARE_DELETE. Nope, still chokes on FILE_SHARE_VALID_FLAGS with access denied. > Another idea would be that the MVFS driver refuses the

Re: MVFS results

2009-07-16 Thread Corinna Vinschen
On Jul 16 13:58, Eric Blake wrote: > Corinna Vinschen cygwin.com> writes: > > For instance, maybe it chokes on the sharing flags. I'd try with > > FILE_SHARE_VALID_FLAGS instead of just FILE_SHARE_DELETE. > > > > Btw., it's very helpful to observe what happens exactly using > > sysinternal's pro

Re: MVFS results

2009-07-16 Thread Eric Blake
Corinna Vinschen cygwin.com> writes: > > Basically, the NtOpenFile took over execution (I'm guessing that it triggered a > > fault handler, which interfered with single stepping?). My next attempt: > > Hmm. > > STATUS_ACCESS_DENIED. But why? > > > Not exactly. If I had MVFS here, I'd try

Re: MVFS results

2009-07-16 Thread Corinna Vinschen
On Jul 16 05:30, Marc Girod wrote: > > > Corinna Vinschen-2 wrote: > > > > Been there. It's a remote trial over Web Browser for 3 hours. > > That's not exactly helpful. Oh well. > > > > The other option is the source code for linux. > It is Open Source. > > I can find it from my release are

Re: MVFS results

2009-07-16 Thread Marc Girod
rived objects: sharing commonalities and managing by difference). I am probably still on the stuff the original developers sniffed... Marc -- View this message in context: http://www.nabble.com/MVFS-results-tp24504520p24515400.html Sent from the Cygwin list mailing list archive at Nabble.com

Re: MVFS results

2009-07-16 Thread Dave Korn
Ralph Hempel wrote: > [ snip ] Sorry, I failed to properly TITTTL the address. Followups set. cheers, DaveK -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentation: http://cygwin.com/docs.html Unsubscribe info:

Re: MVFS results

2009-07-16 Thread Ralph Hempel
Dave Korn wrote: Corinna Vinschen wrote: http://www.ibm.com/developerworks/downloads/r/rcc/?S_TACT=105AGX28&S_CMP=DLMAIN Been there. It's a remote trial over Web Browser for 3 hours. BWAAhahahahhhaahahaa! Remote trial in a web browser? For 3 hours? Are they kidding? It's like som

Re: MVFS results

2009-07-16 Thread Dave Korn
Corinna Vinschen wrote: > On Jul 16 02:19, Andrew DeFaria wrote: >> Corinna Vinschen wrote: >>> How do I get MVFS on my system for testing? >> That would involve getting and installing IBM/Rational's Clearcase and >> get a valid license. They are quite expensive (like $4600 for a single >> lice

Re: MVFS results

2009-07-16 Thread Corinna Vinschen
On Jul 16 02:19, Andrew DeFaria wrote: > Corinna Vinschen wrote: >> How do I get MVFS on my system for testing? > That would involve getting and installing IBM/Rational's Clearcase and > get a valid license. They are quite expensive (like $4600 for a single > license), however, you could proba

Re: MVFS results

2009-07-16 Thread Andrew DeFaria
Corinna Vinschen wrote: How do I get MVFS on my system for testing? That would involve getting and installing IBM/Rational's Clearcase and get a valid license. They are quite expensive (like $4600 for a single license), however, you could probably get an evaluation license. See http://www.i

Re: MVFS results

2009-07-16 Thread Corinna Vinschen
On Jul 15 22:03, Eric Blake wrote: > Corinna Vinschen cygwin.com> writes: > > This is really weird. The R/O flag *is* supported, the SYSTEM flag > > isn't. I assume the HIDDEN flag isn't either, but that's a minor > > problem. > > You are correct, 'attrib +h' has no effect. I also suspect that

Re: MVFS results

2009-07-15 Thread Eric Blake
Corinna Vinschen cygwin.com> writes: > Eric, can you please change syscalls.cc, line 532 from > > NtSetAttributesFile (fh, pc.file_attributes () & ~FILE_ATTRIBUTE_READONLY); > > to > > { > status = NtSetAttributesFile (fh, pc.file_attributes () & ~FILE_ATTRIBUTE_READONLY); > if (!

Re: volinfo (was: MVFS results)

2009-07-15 Thread Corinna Vinschen
On Jul 15 16:54, Christopher Faylor wrote: > On Wed, Jul 15, 2009 at 10:51:29PM +0200, Corinna Vinschen wrote: > >On Jul 15 20:23, Eric Blake wrote: > >> Charles Wilson cwilson.fastmail.fm> writes: > >> > >> > > >> > Eric Blake wrote: > >> > > Also, volinfo is such a helpful debug utility; shoul

Re: volinfo (was: MVFS results)

2009-07-15 Thread Christopher Faylor
On Wed, Jul 15, 2009 at 10:51:29PM +0200, Corinna Vinschen wrote: >On Jul 15 20:23, Eric Blake wrote: >> Charles Wilson cwilson.fastmail.fm> writes: >> >> > >> > Eric Blake wrote: >> > > Also, volinfo is such a helpful debug utility; should we go ahead and >> > > add >> it to >> > > the utils

Re: volinfo (was: MVFS results)

2009-07-15 Thread Corinna Vinschen
On Jul 15 20:23, Eric Blake wrote: > Charles Wilson cwilson.fastmail.fm> writes: > > > > > Eric Blake wrote: > > > Also, volinfo is such a helpful debug utility; should we go ahead and add > it to > > > the utils directory, and compile it alongside other tools like cygcheck > > > as > part

Re: MVFS results

2009-07-15 Thread Corinna Vinschen
On Jul 15 19:42, Eric Blake wrote: > New thread (gmane gave me grief at finding the old one, and anyways, I'm > adding > a couple new topics to the mix). > > $ attrib foo >M:\foo > $ attrib +r +s foo > $ attrib foo > R M:\foo > $ rm foo > rm: remove write-protected regular e

Re: volinfo (was: MVFS results)

2009-07-15 Thread Eric Blake
Eric Blake byu.net> writes: > Oh - that's why I didn't find it - csih renamed it to getVolInfo, and it is > not part of the default PATH. Hmm. Maybe we should teach 'cygcheck -p' to do case-insensitive searches for the regex; this would have turned up csih for 'cygcheck -c volinfo', instead of

volinfo (was: MVFS results)

2009-07-15 Thread Eric Blake
Charles Wilson cwilson.fastmail.fm> writes: > > Eric Blake wrote: > > Also, volinfo is such a helpful debug utility; should we go ahead and add it to > > the utils directory, and compile it alongside other tools like cygcheck as part > > of the base package? > > It's already in the csih pac

Re: MVFS results

2009-07-15 Thread Charles Wilson
Eric Blake wrote: > Also, volinfo is such a helpful debug utility; should we go ahead and add it > to > the utils directory, and compile it alongside other tools like cygcheck as > part > of the base package? It's already in the csih package, under /usr/lib/csih/ but if you want to "promote" i

MVFS results

2009-07-15 Thread Eric Blake
New thread (gmane gave me grief at finding the old one, and anyways, I'm adding a couple new topics to the mix). $ attrib foo M:\foo $ attrib +r +s foo $ attrib foo R M:\foo $ rm foo rm: remove write-protected regular empty file `foo'? y rm: cannot remove `foo': Permission den