public inbox for cygwin-patches@cygwin.com
 help / color / mirror / Atom feed
* Add support for Windows 8, first step
@ 2011-10-11 19:39 Teemu Nätkinniemi
  2011-10-12  8:27 ` Corinna Vinschen
  0 siblings, 1 reply; 3+ messages in thread
From: Teemu Nätkinniemi @ 2011-10-11 19:39 UTC (permalink / raw)
  To: cygwin-patches

[-- Attachment #1: Type: text/plain, Size: 303 bytes --]

Hello!

Here's a small patch that enables Cygwin to run on Windows 8 (tested on 
x64 build 8102, the Windows Developer Preview). Windows 8 does not seem 
to support FAST_CWD or the current implementation of FAST_CWD is not 
compatible with Windows 8 so it is disabled at the moment.

Teemu Nätkinniemi

[-- Attachment #2: wincap_win8.patch --]
[-- Type: text/plain, Size: 1591 bytes --]

2011-10-11  Teemu Nätkinniemi  <tnatkinn@gmail.com>

	* wincap.cc (wincap_8): Add support to Windows 8
	(wincapc::init): Ditto

--- wincap.cc.orig	2011-07-30 23:51:03.000000000 +0300
+++ wincap.cc	2011-10-11 22:13:44.556795800 +0300
@@ -263,6 +263,36 @@
   has_stack_size_param_is_a_reservation:true,
 };
 
+wincaps wincap_8 __attribute__((section (".cygwin_dll_common"), shared)) = {
+  max_sys_priv:SE_CREATE_SYMBOLIC_LINK_PRIVILEGE,
+  is_server:false,
+  has_physical_mem_access:false,
+  has_create_global_privilege:true,
+  has_ioctl_storage_get_media_types_ex:true,
+  has_disk_ex_ioctls:true,
+  has_buggy_restart_scan:false,
+  has_mandatory_integrity_control:true,
+  needs_logon_sid_in_sid_list:false,
+  needs_count_in_si_lpres2:false,
+  has_recycle_dot_bin:true,
+  has_gaa_prefixes:true,
+  has_gaa_on_link_prefix:true,
+  supports_all_posix_ai_flags:true,
+  has_restricted_stack_args:false,
+  has_transactions:true,
+  has_recvmsg:true,
+  has_sendmsg:true,
+  has_broken_udf:false,
+  has_console_handle_problem:true,
+  has_broken_alloc_console:true,
+  has_always_all_codepages:true,
+  has_localenames:true,
+  has_fast_cwd:false,
+  has_restricted_raw_disk_access:true,
+  use_dont_resolve_hack:false,
+  has_stack_size_param_is_a_reservation:true,
+};
+
 wincapc wincap __attribute__((section (".cygwin_dll_common"), shared));
 
 void
@@ -320,9 +350,12 @@
 		  case 0:
 		    caps = &wincap_vista;
 		    break;
-		  default:
+		  case 1:
 		    caps = &wincap_7;
 		    break;
+		  default:
+		    caps = &wincap_8;
+		    break;
 		}
 	      break;
 	    default:

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Add support for Windows 8, first step
  2011-10-11 19:39 Add support for Windows 8, first step Teemu Nätkinniemi
@ 2011-10-12  8:27 ` Corinna Vinschen
  2011-10-12 20:45   ` Teemu Nätkinniemi
  0 siblings, 1 reply; 3+ messages in thread
From: Corinna Vinschen @ 2011-10-12  8:27 UTC (permalink / raw)
  To: cygwin-patches

Hi Teemu,

First of all, we need a copyright assignment from you before we can
accept non-trivial patches to Cygwin, see http://cygwin.com/contrib.html,
the "Before you get started" section.

On Oct 11 22:38, Teemu Nätkinniemi wrote:
> Hello!
> 
> Here's a small patch that enables Cygwin to run on Windows 8 (tested
> on x64 build 8102, the Windows Developer Preview). Windows 8 does
> not seem to support FAST_CWD or the current implementation of
> FAST_CWD is not compatible with Windows 8 so it is disabled at the
> moment.

Windows 8 will very likely support the FAST_CWD stuff, the problem is
just to find out how to find the global pointer pointing to the current
FAST_CWD structure, and then, if the FAST_CWD structure changed.

Having this annoying message at startup was intentional, so that we
know that there's some work to do yet.  However, personally I don't plan
to look into W8 so soon.  Stuff like that is bound to change this early
in the release game anyway.

Therefore I don't want to disable this message.  If you're interested
to get rid of it, it would be most helpful trying to track down how to
find the global FAST_CWD pointer in W8.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Project Co-Leader          cygwin AT cygwin DOT com
Red Hat

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Add support for Windows 8, first step
  2011-10-12  8:27 ` Corinna Vinschen
@ 2011-10-12 20:45   ` Teemu Nätkinniemi
  0 siblings, 0 replies; 3+ messages in thread
From: Teemu Nätkinniemi @ 2011-10-12 20:45 UTC (permalink / raw)
  To: cygwin-patches

Hello Corinna,

On 12.10.2011 11:26, Corinna Vinschen wrote:

> First of all, we need a copyright assignment from you before we can
> accept non-trivial patches to Cygwin, see http://cygwin.com/contrib.html,
> the "Before you get started" section.

I am going to do that just in case I actually get something working (see 
below).

> Windows 8 will very likely support the FAST_CWD stuff, the problem is
> just to find out how to find the global pointer pointing to the current
> FAST_CWD structure, and then, if the FAST_CWD structure changed.

It does do that, the code looks almost the same as in Windows 7. There 
are some differences that I haven't figured out yet. But I might 
actually wait for a beta version of Windows 8 before going forward.

> Therefore I don't want to disable this message.  If you're interested
> to get rid of it, it would be most helpful trying to track down how to
> find the global FAST_CWD pointer in W8.

If I understood the code correctly f_cwd_ptr is the location of 
ntdll!RtlpCurDirRef and find_fast_cwd_pointer tries to find that 
location. For some reason I couldn't get breakpoints working when 
debugging cygwin1.dll so I did some disassembling and found the correct 
location for ntdll!RtlpCurDirRef in Windows 8 version of ntdll (wow64).

Any hints for debugging the Cygwin dll itself as find_fast_cwd runs once 
per session so it has been very difficult trying to get a working 
breakpoint.

Teemu

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-10-12 20:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-10-11 19:39 Add support for Windows 8, first step Teemu Nätkinniemi
2011-10-12  8:27 ` Corinna Vinschen
2011-10-12 20:45   ` Teemu Nätkinniemi

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 read-only IMAP folder(s) and NNTP newsgroup(s).